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.

Sign up for GitHub

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

Jump to bottom

Paginate channels to get over 200 channels limit #1

Merged
merged 1 commit into from
Nov 6, 2022
Merged

Paginate channels to get over 200 channels limit #1

merged 1 commit into from
Nov 6, 2022

Conversation

42wim
Copy link
Member

@42wim 42wim commented Nov 5, 2022

This was referenced Nov 5, 2022
Closed
Merged
@hloeung
Copy link
Contributor

hloeung commented Nov 5, 2022

@42wim LGTM. Also cleaner fix, thanks.

One thing, can we make the found X messages INFO?

[2022-11-05T21:04:16Z]  INFO matterclient: Found version 6.6.0.6.6.0.96a259e895e1143a8e843b070c21019a.true
[2022-11-05T21:04:21Z] DEBUG matterclient: found 947 users in team canonical
[2022-11-05T21:04:21Z] DEBUG matterclient: initUser(): found our team canonical (id: sqmc4sz45prypmkfctwynm5yjr)
[2022-11-05T21:04:26Z] DEBUG matterclient: found 158 channels for user in team canonical
[2022-11-05T21:04:26Z] DEBUG matterclient: found 1316 public channels in team canonical

I think those are useful or nice to see.

@42wim 42wim merged commit e65d60f into master Nov 6, 2022
@42wim 42wim deleted the fixchannels branch November 6, 2022 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@42wim @hloeung