Skip to content

Handle missing members in XP lookup - #1701

Open
itzMiney wants to merge 3 commits into
r-Techsupport:mainfrom
itzMiney:xp-top-patch
Open

itzMiney wants to merge 3 commits into
r-Techsupport:mainfrom
itzMiney:xp-top-patch

Conversation

@itzMiney

Copy link
Copy Markdown
Member

Refactored /xp top command to fall back to the global user object if a member can't be found for a db record

This little patch is to fix issue #1700

Built and ran the bot on my test server and the patch seems to work

image

Feel free to leave some feedback if anything needs changing

Refactored /xp top command to fall back to the global user object if a member can't be found for a db record
I forgot to remove the import...
@itzMiney

Copy link
Copy Markdown
Member Author

yk it is kinda just warnings not errors, but it bothers me so I fix what the checks say bcs yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant