Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Qstick
Copy link
Contributor

@Qstick Qstick commented Sep 9, 2019

Database Migration

YES | NO

Description

A few sentences describing the overall goals of the pull request's commits.

Todos

  • Tests
  • Documentation

Issues Fixed or Closed by this PR

Qstick and others added 30 commits April 11, 2019 21:15
Wrap any exception thrown in a LidarrStartupException to trigger the
existing bootloop code.

We have to catch the generic Exception class since FluentMigrator
rethrows all exceptions as a generic Exception.
Fixes error in manual import page when Tracks is missing.

Fixes Sentry LIDARR-UI-1GC
* Fixed: Artist only marked as refreshed if Albums/Tracks refreshed also

* Fixed: Skip album releases with no valid tracks

* Fixed: Don't add albums with no valid releases

* Fixed: Remove Albums with no valid releases
Qstick and others added 28 commits August 25, 2019 01:20
* New: Unmapped files view

Displays all trackfiles that haven't been matched to a track.
Generalised the file details component and adds it to the album
details screen.

* Add sorting by quality

* New: MediaServiceTests & MediaRepoTests
* New: Swap to ImageSharp for image resizing to avoid leaks

Stop resizing album images also

* Fixed: MediaCoverModule falls back to fullsize for png and gif too

* Fixed: Look for all image extensions in DeleteBadMediaCovers.cs
It fails due to requring secrets not available on forked PRs
Simplify entity equality code and enfore db/metadata split

Use a nuget package to remove boilerplate code that needs careful
update when adding/removing fields.  Add tests to enforce that all
fields are allocated to 'UseMetadataFrom' or 'UseDbFieldsFrom' to make
metadata refresh more foolproof.

Fix NRE when tracks are merged because artist wasn't set.
Fix NRE when tracks are merged and the merge target wasn't yet in the database.
It's not crucial so if it fails don't take out the whole import process.
…Also bumped minimum mono version check to 5.16 (5.20 is the best choice atm)
* Centralize version numbers

* Fixed: Run UpdateVersionNumber on Package (For Separate Jobs)
@Qstick Qstick merged commit dbf8131 into master Sep 9, 2019
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants