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

feat: Add Auto Approve 4K role #4982

Merged
merged 1 commit into from
Jul 14, 2023

Conversation

sephrat
Copy link
Contributor

@sephrat sephrat commented Jul 14, 2023

Legacy "Auto Approve" role now only applies to non-4K requests
Fixes #4957

Legacy "Auto Approve" role now only applies to non-4K requests
Fixes Ombi-app#4957
@tidusjar
Copy link
Member

For some reason the Automation tests are failing sometimes trying to authenticate.
Going to re-run them

@tidusjar
Copy link
Member

Looks like the tests are running out of order. Unfortunatly the test suite always requires the Wizard to be setup first on a new installation. I'm going to merge this into a autoapprove4k branch and see what's going on

@tidusjar tidusjar changed the base branch from develop to autoapprove4k July 14, 2023 13:44
@tidusjar tidusjar merged commit 3e19395 into Ombi-app:autoapprove4k Jul 14, 2023
11 of 14 checks passed
tidusjar added a commit that referenced this pull request Jul 14, 2023
Legacy "Auto Approve" role now only applies to non-4K requests
Fixes #4957

Co-authored-by: sephrat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Users without 4K permissions can request 4K movies
2 participants