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
Migrated cheermotes to Helix API #2440
Conversation
e4409d0
to
bbc9e72
Compare
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.
I haven't tested qmake builds, but if that works I'm good with this change 👌 Merge when you've had a change to review/approve my additions @zneix
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.
The pull request seems fine, thank you so much pajlada for finishing the parsing in it.
The only thing to note for the future is that naming of cheermotes is now inconsistent in the codebase as old code still calls them "cheerEmotes".
* chatterino/master: Add setting to disable custom ffz mod/VIP badges (Chatterino#2759) v2.3.2 added try catch in nm (Chatterino#2785) Fixed deprecated method Qt::SystemLocaleLongDate (Chatterino#2784) Migrated cheermotes to Helix API (Chatterino#2440) Fix word backspacing not closing/updating colon emote picker (Chatterino#2781) Load 100 blocked users instead of default 20 (Chatterino#2772) Refactored and Migrated to Helix AutoMod message management (Chatterino#2779) Add flatpak version badge (Chatterino#2782) Bump dangoslen/changelog-enforcer from 2.1.0 to 2.2.0 (Chatterino#2770) Bump ilammy/msvc-dev-cmd from 1.7.0 to 1.8.0 (Chatterino#2769)
Release v2.3.2 - Major: New split for channels going live! /live. (Chatterino#1797) - Minor: Added a message that displays a new date on new day. (Chatterino#1016) - Minor: Hosting messages are now clickable. (Chatterino#2655) - Minor: Messages held by automod are now shown to the user. (Chatterino#2626) - Minor: Load 100 blocked users rather than the default 20. (Chatterino#2772) - Bugfix: Fixed a potential crashing issue related to the browser extension. (Chatterino#2774) - Bugfix: Strip newlines from stream titles to prevent text going off of split header (Chatterino#2755) - Bugfix: Automod messages now work properly again. (Chatterino#2682) - Bugfix: `Login expired` message no longer highlights all tabs. (Chatterino#2735) - Bugfix: Fix a deadlock that would occur during user badge loading. (Chatterino#1704, Chatterino#2756) - Bugfix: Tabbing in `Select a channel to open` is now consistent. (Chatterino#1797) - Bugfix: Fix Ctrl + Backspace not closing colon emote picker. (Chatterino#2780) - Bugfix: Approving/denying AutoMod messages works again. (Chatterino#2779) - Dev: Migrated AutoMod approve/deny endpoints to Helix. (Chatterino#2779) - Dev: Migrated Get Cheermotes endpoint to Helix. (Chatterino#2440) # gpg: directory '/c/Users/gtroy/.gnupg' created # gpg: keybox '/c/Users/gtroy/.gnupg/pubring.kbx' created # gpg: Signature made Sun May 16 15:05:25 2021 EDT # gpg: using RSA key 16CA5983F9FDD6EB599AAA70911916E0523B22F6 # gpg: Can't check signature: No public key # Conflicts: # src/widgets/settingspages/GeneralPage.cpp
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
Migrates Kraken's Get Cheermotes to its Helix equivalent.
(Finally) Closes #1928