fix: general settings min/max filesize settings #502
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix: min/max filesize being returned undefined
iPromKnight
added a commit
to iPromKnight/riven
that referenced
this pull request
Jul 13, 2024
* testing * testing * minor fixes * feat: some frontend ui changes :) * minor tweaks * added movies and top movies section * nearly completed homepage * fix: fix settings and improvements to homepage * chore: update app name to Riven and fix page header in library and onboarding * feat: switch to vaul-svelte dependency for mobile ui and improvements to ui in general * chore: update app name to Riven, improve UI consistency and changes to items endpoint * feat: Add incomplete items to statistics page * feat: Add services status to statistics page feat: add lazy loading for images in statistics and home pages (rivenmedia#502) fix: min/max filesize being returned undefined fix: minor ui improvements (rivenmedia#503) * fix: minor ui improvements * chore: formatted files fix: lower the z index and increase z index of header (rivenmedia#504) feat: add top rated section (rivenmedia#505) * feat: add top rated section * chore: format files fix: text color on light theme (rivenmedia#506) chore(deps-dev): bump vitest from 1.6.0 to 2.0.1 in /frontend (rivenmedia#498) Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 1.6.0 to 2.0.1. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.1/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Rate limiting for RD - Change requests models to take through an optional instance of the rate limiter class - Implement rate limiter settings for RD, a Conservative 100 calls per 60 seconds as default - Ensure if enabled the rate limiter is initialised within the RD class - Ensure all Rd calls (apart from ping) use the initialised rate limiter if present in requests Add RD rate limit settings to general settings pane - frontend Ignore mac and rider specific settings remove config from frontend - this is internal now, non configurable remove config, and use oll the time 1 per second Split out rate limiter to seprate file, also work in 429's and handle overall rate limiting and endpoint specific ratelimiting Use get for ping, and pass rate limiters Added a user agent factory, and randomiser in requests Updated zilean scraper to filtered endpoint Only on movies. Aired at wont be right for seasons, we need a better way to do this across the board
dreulavelle
pushed a commit
that referenced
this pull request
Jul 13, 2024
* testing * testing * minor fixes * feat: some frontend ui changes :) * minor tweaks * added movies and top movies section * nearly completed homepage * fix: fix settings and improvements to homepage * chore: update app name to Riven and fix page header in library and onboarding * feat: switch to vaul-svelte dependency for mobile ui and improvements to ui in general * chore: update app name to Riven, improve UI consistency and changes to items endpoint * feat: Add incomplete items to statistics page * feat: Add services status to statistics page feat: add lazy loading for images in statistics and home pages (#502) fix: min/max filesize being returned undefined fix: minor ui improvements (#503) * fix: minor ui improvements * chore: formatted files fix: lower the z index and increase z index of header (#504) feat: add top rated section (#505) * feat: add top rated section * chore: format files fix: text color on light theme (#506) chore(deps-dev): bump vitest from 1.6.0 to 2.0.1 in /frontend (#498) Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 1.6.0 to 2.0.1. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.1/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Rate limiting for RD - Change requests models to take through an optional instance of the rate limiter class - Implement rate limiter settings for RD, a Conservative 100 calls per 60 seconds as default - Ensure if enabled the rate limiter is initialised within the RD class - Ensure all Rd calls (apart from ping) use the initialised rate limiter if present in requests Add RD rate limit settings to general settings pane - frontend Ignore mac and rider specific settings remove config from frontend - this is internal now, non configurable remove config, and use oll the time 1 per second Split out rate limiter to seprate file, also work in 429's and handle overall rate limiting and endpoint specific ratelimiting Use get for ping, and pass rate limiters Added a user agent factory, and randomiser in requests Updated zilean scraper to filtered endpoint Only on movies. Aired at wont be right for seasons, we need a better way to do this across the board
iPromKnight
added a commit
to iPromKnight/riven
that referenced
this pull request
Jul 22, 2024
* testing * testing * minor fixes * feat: some frontend ui changes :) * minor tweaks * added movies and top movies section * nearly completed homepage * fix: fix settings and improvements to homepage * chore: update app name to Riven and fix page header in library and onboarding * feat: switch to vaul-svelte dependency for mobile ui and improvements to ui in general * chore: update app name to Riven, improve UI consistency and changes to items endpoint * feat: Add incomplete items to statistics page * feat: Add services status to statistics page feat: add lazy loading for images in statistics and home pages (rivenmedia#502) fix: min/max filesize being returned undefined fix: minor ui improvements (rivenmedia#503) * fix: minor ui improvements * chore: formatted files fix: lower the z index and increase z index of header (rivenmedia#504) feat: add top rated section (rivenmedia#505) * feat: add top rated section * chore: format files fix: text color on light theme (rivenmedia#506) chore(deps-dev): bump vitest from 1.6.0 to 2.0.1 in /frontend (rivenmedia#498) Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 1.6.0 to 2.0.1. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.1/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Rate limiting for RD - Change requests models to take through an optional instance of the rate limiter class - Implement rate limiter settings for RD, a Conservative 100 calls per 60 seconds as default - Ensure if enabled the rate limiter is initialised within the RD class - Ensure all Rd calls (apart from ping) use the initialised rate limiter if present in requests Add RD rate limit settings to general settings pane - frontend Ignore mac and rider specific settings remove config from frontend - this is internal now, non configurable remove config, and use oll the time 1 per second Split out rate limiter to seprate file, also work in 429's and handle overall rate limiting and endpoint specific ratelimiting Use get for ping, and pass rate limiters Added a user agent factory, and randomiser in requests Updated zilean scraper to filtered endpoint Only on movies. Aired at wont be right for seasons, we need a better way to do this across the board
iPromKnight
added a commit
to iPromKnight/riven
that referenced
this pull request
Jul 22, 2024
* testing * testing * minor fixes * feat: some frontend ui changes :) * minor tweaks * added movies and top movies section * nearly completed homepage * fix: fix settings and improvements to homepage * chore: update app name to Riven and fix page header in library and onboarding * feat: switch to vaul-svelte dependency for mobile ui and improvements to ui in general * chore: update app name to Riven, improve UI consistency and changes to items endpoint * feat: Add incomplete items to statistics page * feat: Add services status to statistics page feat: add lazy loading for images in statistics and home pages (rivenmedia#502) fix: min/max filesize being returned undefined fix: minor ui improvements (rivenmedia#503) * fix: minor ui improvements * chore: formatted files fix: lower the z index and increase z index of header (rivenmedia#504) feat: add top rated section (rivenmedia#505) * feat: add top rated section * chore: format files fix: text color on light theme (rivenmedia#506) chore(deps-dev): bump vitest from 1.6.0 to 2.0.1 in /frontend (rivenmedia#498) Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 1.6.0 to 2.0.1. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.1/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Rate limiting for RD - Change requests models to take through an optional instance of the rate limiter class - Implement rate limiter settings for RD, a Conservative 100 calls per 60 seconds as default - Ensure if enabled the rate limiter is initialised within the RD class - Ensure all Rd calls (apart from ping) use the initialised rate limiter if present in requests Add RD rate limit settings to general settings pane - frontend Ignore mac and rider specific settings remove config from frontend - this is internal now, non configurable remove config, and use oll the time 1 per second Split out rate limiter to seprate file, also work in 429's and handle overall rate limiting and endpoint specific ratelimiting Use get for ping, and pass rate limiters Added a user agent factory, and randomiser in requests Updated zilean scraper to filtered endpoint Only on movies. Aired at wont be right for seasons, we need a better way to do this across the board
dreulavelle
added a commit
that referenced
this pull request
Jul 22, 2024
* Release 080 (#493) * Start SQLAlchemy intgr * RE: #2 .. Save Work * RE: #2 sync changes * re: #2 end of day sync * Re: #2 , Finish program outline and replace core components. * RE: #2 sync * Re: #2 still some known issues * Re: #2 - Running Alpha - V1 * Re: #2 Remove final known item_id.parent_id * Re: #2 Something's happening! * RE: #3 alpha-2 * Re: #3 final sync all changes * Re: #2 Fix failed symlinker to not always immediately yield the item back --------- Co-authored-by: Administrator <[email protected]> * fix: tidy. removed clean_title. * fix(deps): replace psycopg2 with psycopg2-binary * chore: added libpq-dev dep to dockerfile * Various fixes (#509) * Fix: Actually add new items to the database and back to queue * Fix: Pass existing item to state_transition if one exists * feat: add critical error message and fail out when trying to enhance the same item due to multiple folder entries * Fix: add the item_id to the added array so the enhancement error will trigger. * Fix: Don't expunge the objects --------- Co-authored-by: Administrator <[email protected]> * Fix: Ensure streams object is recreated when loading from database and set to a default value and remove the static instance from the class definition. (#513) Co-authored-by: Administrator <[email protected]> * Release 080 patches (#515) * Fix: Clarify log message about backoff timer being hit from the scraper. * Tidy: Remove pre-loading of children and setting their parents Fix: Always set the streams from the old input item on calls to _get_item_from_db Fix: Remove redundant copy streams Feat: Add HARD_RESET env variable to erase the database on start * Tidy: Remove always print HARD_RESET value --------- Co-authored-by: Administrator <[email protected]> * Comet Scraper (#511) * Add first version of comet scraper Fix annatar not using custom url for validation * Fix imdb id in comet scraper * Finish comet scraper * feat: improved ui (#422) (#512) * testing * testing * minor fixes * feat: some frontend ui changes :) * minor tweaks * added movies and top movies section * nearly completed homepage * fix: fix settings and improvements to homepage * chore: update app name to Riven and fix page header in library and onboarding * feat: switch to vaul-svelte dependency for mobile ui and improvements to ui in general * chore: update app name to Riven, improve UI consistency and changes to items endpoint * feat: Add incomplete items to statistics page * feat: Add services status to statistics page feat: add lazy loading for images in statistics and home pages (#502) fix: min/max filesize being returned undefined fix: minor ui improvements (#503) * fix: minor ui improvements * chore: formatted files fix: lower the z index and increase z index of header (#504) feat: add top rated section (#505) * feat: add top rated section * chore: format files fix: text color on light theme (#506) chore(deps-dev): bump vitest from 1.6.0 to 2.0.1 in /frontend (#498) Bumps [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) from 1.6.0 to 2.0.1. - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.1/packages/vitest) --- updated-dependencies: - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Rate limiting for RD - Change requests models to take through an optional instance of the rate limiter class - Implement rate limiter settings for RD, a Conservative 100 calls per 60 seconds as default - Ensure if enabled the rate limiter is initialised within the RD class - Ensure all Rd calls (apart from ping) use the initialised rate limiter if present in requests Add RD rate limit settings to general settings pane - frontend Ignore mac and rider specific settings remove config from frontend - this is internal now, non configurable remove config, and use oll the time 1 per second Split out rate limiter to seprate file, also work in 429's and handle overall rate limiting and endpoint specific ratelimiting Use get for ping, and pass rate limiters Added a user agent factory, and randomiser in requests Updated zilean scraper to filtered endpoint Only on movies. Aired at wont be right for seasons, we need a better way to do this across the board * fix: add missing os import * Feature/alldebrid support (#517) * All Debrid Support (backend) * Frontend support for alldebrid settings * Refactor Zilean scraper for MediaItem support (#516) The Zilean scraper has been refactored to support the MediaItem class. Previously, it only supported the Movie class. The 'aired_at' attribute of a Movie instance is now replaced with the 'year' attribute of a MediaItem instance. This change allows more flexibility in handling different types of media items beyond just movies. * fix: add check for Overseerr pageInfo attr on results * fix: check results length appropriately on overseerr * fix: added check for `None` filenames during symlinking * fix: tidy initialize on fresh start * fix: downloader init validation and run fixed. none.mkv fixed from symlinking * Settings migration (#527) * Settings migration Migrate settings from older versions to new version on run. Saves settings after model assignment * Ensure db config is written * DB Migrations Path - use utils data dir. (#533) * Settings migration Migrate settings from older versions to new version on run. Saves settings after model assignment * Ensure db config is written * hotfix db migration path * fix: fix wanted shows in rd. fixed /stats endpoint * fix: temporarily disable blacklisting from RD * Fix yield of plex watchlist (#538) * fix: fixed bug with /services endpoint with downloaders init * fix: bandaid frontend endpoints for now * chore!: renamed backend dir to src. updated Dockerfile * fix: out with old :) * fix: update Dockerfile.slim to use src dir * fix: update entrypoint to src dir * fix: update Dockerfile to use src dir * fix: update workflow and makefile, and coverage * tidy: add logging to rd. fix items controller. * Fixes: better fill in enchanced mediaitems from symlinks and modify program to use id for queue checking (#541) * Fix: media-enhancer creates blank items for missing seasons and episodes * Fix: Check for items in queue by _id instead of comparing objects to prevent issues. * Fix: Remove erronous returns * Feat: Try and scrape entire show/season again if backoff timer allows when in partially completed * Fix: Allow shows and seasons to return scraped state if they're scraped so they can proceed to the next phase --------- Co-authored-by: Administrator <[email protected]> * feat/Torbox cached downloader support * fix: removed cache entirely. loosened up rd packs to accept any episode that matches instead of requiring full packs * fix: plex watchlist has been fixed * Remove frontend * Remove frontend from docker (#534) --------- Co-authored-by: Matthew Adams <[email protected]> Co-authored-by: Administrator <[email protected]> Co-authored-by: Dreu LaVelle <[email protected]> Co-authored-by: Davide Marcoli <[email protected]> Co-authored-by: Gaisberg <None>
This was referenced Jul 24, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix: min/max filesize being returned undefined
Pull Request Check List
Resolves: #issue-number-here
Description: