-
-
Notifications
You must be signed in to change notification settings - Fork 442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show all global FFZ badges of a user #3818
Conversation
I'm not sure if this is the correct approach, the FFZ Bot badges we load in Chatterino are a relic of the past and purely appear because they are in the same API as supporter badges. |
This doesn't really conflict with the transition to v2 since I don't think anyone is working on that right now, so I'm happy to merge this feature in. Will push a little commit cleaning up the code at the same time as I solve the merge conflict, then let @Jaxkey get the last review in |
Only expose a "getUserBadges" that returns a vector of badges, which can be empty for users with no badges
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected 👍
Happy to merge upon your approval of my changes @Jaxkey
Looks good @pajlada. Merge it 👍 |
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
In Chatterino, a maximum of one global FFZ badge appears at a time.
Moobot has both the
Bot
&FFZ Supporter
global FFZ Badges.In Chatterino, the
Bot
badge is missing and only one FFZ badge appears.Changed
badgeMap
to store a vector of ID's in its second position for each FFZ badge a user has (previously only stored a single ID).Changed
appendFfzBadges()
to loop through each FFZ badge a user has.Updated version:
To Reproduce
Have a user with multiple global FFZ badges send a chat message