Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix trending count #1103

Merged
merged 1 commit into from
Feb 9, 2024
Merged

Fix trending count #1103

merged 1 commit into from
Feb 9, 2024

Conversation

Ben3056
Copy link
Contributor

@Ben3056 Ben3056 commented Jan 5, 2024

The existing implementation always returns 30 videos. This fixes that issue to perform as suggested

@davidteather davidteather changed the base branch from main to v6.2.1 February 9, 2024 15:32
@davidteather davidteather merged commit 9f2d222 into davidteather:v6.2.1 Feb 9, 2024
davidteather added a commit that referenced this pull request Feb 9, 2024
* remove sponsors

* Fix count parameter for hashtag, user, and trending classes (#1108)

* Fix trending count (#1103)

Co-authored-by: ben.wisnet <ben.wisnet@gmail.com>

* Made author field optional (#1099)

Fixed issue where error is raised for some videos that are missing the "author" field

* bump version

---------

Co-authored-by: lysmm203 <65572921+lysmm203@users.noreply.github.com>
Co-authored-by: Ben3056 <32847329+Ben3056@users.noreply.github.com>
Co-authored-by: ben.wisnet <ben.wisnet@gmail.com>
Co-authored-by: Tristan Lee <lee.tristan.evans@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants