Skip to content
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

Load 100 blocked users instead of default 20 #2772

Merged
merged 3 commits into from May 14, 2021

Conversation

iProdigy
Copy link
Contributor

@iProdigy iProdigy commented May 12, 2021

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

Work towards #2767 so that at least up to 100 blocked users are returned rather than the default 20

@fourtf
Copy link
Member

fourtf commented May 12, 2021

For some wild reason the "count" is actually called "first". Haven't tested but should work.

https://dev.twitch.tv/docs/api/reference#get-user-block-list

@zneix
Copy link
Collaborator

zneix commented May 13, 2021

I think that following pagination to load all the blocks is a better solution, however this is better than nothing.

@jupjohn
Copy link
Contributor

jupjohn commented May 14, 2021

Might be nicer to paginate the blocked users page so that we can return a decent chunk. Might be too bloated for current settings layout though

@pajlada
Copy link
Member

pajlada commented May 14, 2021

I think that following pagination to load all the blocks is a better solution, however this is better than nothing.

Pagination would definitely be what we want to do, but even with a pagination PR we'd want to increase the limit to 100 so this is a good first step

Copy link
Collaborator

@zneix zneix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair point, I agree with paj.

@zneix zneix changed the title feat(helix): load 100 blocked users rather than 20 Load 100 blocked users instead of default 20 May 14, 2021
@zneix zneix enabled auto-merge (squash) May 14, 2021 14:32
@zneix zneix disabled auto-merge May 14, 2021 15:41
@zneix zneix enabled auto-merge (squash) May 14, 2021 18:15
@zneix zneix merged commit 860cc89 into Chatterino:master May 14, 2021
@iProdigy iProdigy deleted the feature/helix-blocks-100 branch May 14, 2021 19:06
vaehx added a commit to vaehx/chatterino2 that referenced this pull request May 16, 2021
* 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)
GabeEddyT added a commit to GabeEddyT/chatterino2 that referenced this pull request May 25, 2021
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
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.

None yet

5 participants