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

fix: peertube sends too many view requests #836

Merged
merged 2 commits into from
Dec 21, 2022
Merged

fix: peertube sends too many view requests #836

merged 2 commits into from
Dec 21, 2022

Conversation

maxgithubprofile
Copy link
Collaborator

@maxgithubprofile maxgithubprofile commented Dec 21, 2022 •
edited by shpingalet007
Loading

Standards checklist:

  • The PR title is descriptive.
  • The PR doesn't replicate another PR which is already open.
  • The PR has self-explained commits history.
  • The code is mine or it's from somewhere with an Apache-2.0 compatible license.
  • The code is efficient, to the best of my ability, and does not waste computer resources.
  • The code is stable and I have tested it myself, to the best of my abilities.
  • If the code introduces new classes, methods, I provide a valid use case for all of them.

Changes:

  • [...]

Other comments:

...

@auto-assign auto-assign bot requested a review from shpingalet007 December 21, 2022 20:04
@maxgithubprofile maxgithubprofile merged commit 553f4c4 into master Dec 21, 2022
@shpingalet007
Copy link
Collaborator

@andyoknen, this PR hasn't self-described commit history and has not-prefixed title. Please, next time be more careful with reviews.

@maxgithubprofile, please follow the rules of PR's described in template.

@shpingalet007 shpingalet007 added the contribution type: 🐛 fix Something isn't working label Dec 22, 2022
@shpingalet007 shpingalet007 added this to the Next release milestone Dec 22, 2022
@shpingalet007 shpingalet007 changed the title USER_VIEW_VIDEO_INTERVAL: 120000 fix: peertube sends too many view requests Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andyoknen andyoknen andyoknen approved these changes

@shpingalet007 shpingalet007 Awaiting requested review from shpingalet007

Assignees

@maxgithubprofile maxgithubprofile

Labels
contribution type: 🐛 fix Something isn't working
Projects
Team Management
Status: ✅ Done
Milestone
Next release
Development

Successfully merging this pull request may close these issues.
3 participants
@maxgithubprofile @shpingalet007 @andyoknen