* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
When an artist is not found in Lidarr as part of requesting an album,
MusicSender will make a POST request against the /api/v1/artist endpoint
to add such artist.
Not all fields defined in ArtistAdd are initialized, and those
uninitialized will be `null` values in the JSON body of the request, as
shown in this intercepted request:
```
{
"addOptions": {
"AlbumsToMonitor": [
"e5c48b66-44ef-3685-ad53-45dbcd7294c0"
],
"monitor": 6,
"monitored": true,
"searchForMissingAlbums": false
},
"added": "2022-08-10T06:49:32.4374278+00:00",
"albumFolder": true,
"artistName": "Manolo García",
"cleanName": "manologarcía",
"disambiguation": null,
"discogsId": 0,
"ended": false,
"foreignArtistId": "1c8309da-9789-40bf-b9c2-e20064263820",
"images": [],
"links": [],
"metadataProfileId": 1,
"monitored": true,
"overview": null,
"qualityProfileId": 3,
"ratings": null,
"remotePoster": null,
"rootFolderPath": "/media/music/",
"sortName": null,
"statistics": null,
"status": null,
"tadbId": 0,
"tags": null
}
```
This request will fail and Lidarr will return a 400 BadRequest error
with the following message:
```
2022-08-10 01:45:52.458 +00:00 [Error] StatusCode: BadRequest, Reason: Bad Request, RequestUri: http://lidarr:8686/api/v1/artist
2022-08-10 01:45:52.459 +00:00 [Debug] {
"type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
"title": "One or more validation errors occurred.",
"status": 400,
"traceId": "00-52e01b322a05d7c6633eca2488ef2a5c-06345b3bb8c4bb6c-00",
"errors": {
"$.status": [
"The JSON value could not be converted to NzbDrone.Core.Music.ArtistStatusType. Path: $.status | LineNumber: 0 | BytePositionInLine: 14."
]
}
}
```
Removing all the `null` fields from the JSON body fixes the problem and
correctly adds the artist to Lidarr.
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* Remove dead code
* Localize TV requests messages on TV details page
* Transform buttons with link into anchors
* Sonarr sync: stop using seasonpass API
* chore(release): 🚀 v4.16.13
* Fix requests when 4k available and 4k disabled
Fixes#4610
* chore(release): 🚀 v4.16.14
* Hide subscribe button when request is available
* Hide subscribe button when request is denied
* Add Title to Partially Available Message
If the Title of the show is not menitoned it can be unclear what Episodes are now available.
* Better error message when test email fails due to missing recipient
* feat(discover): Add original language filter
* chore(release): 🚀 v4.16.15
* fix(4616): 🐛 fixed mandatory fields
* chore: 👥 Updated Contributors [skip ci]
* chore(release): 🚀 v4.16.16
* added test results into the PR pipeline
* chore(release): 🚀 v4.16.17
* Add information about cache refresh
* Update pr.yml
[skip ci]
* Update pr.yml
[skip ci]
* Update pr.yml
[skip ci]
* chore(release): 🚀 v4.17.0
* feat(discover): Add new trending source experimental feature
* fix(settings): Allow toggling features when there are more than one
* fix(discover): Fix new trending feature detection
* fix(discover): Fix cache mix up
* refactor(discover): Move movie trending feature toggle to backend
* feat(discover): Default trending source to new logic
* chore(release): 🚀 v4.18.0
* feat(sync): Detect reidentified movies in Emby and Jellyfin
* feat(sync): Detect reidentified series in Emby and Jellyfin
* Fix sync log criticity
* Update pr.yml
[skip ci]
* Update label.yml
[skip ci]
* Fix formatting
* Update pr.yml
[skip ci]
* Update label.yml
[skip ci]
* chore: 👥 Updated Contributors [skip ci]
* chore(release): 🚀 v4.19.0
* refactor(newsletter): Clarify very rare cases where newsletter doesn't publish a series
* refactor(newsletter): Clarify very rare cases where newsletter doesn't publish movie
* chore(release): 🚀 v4.19.1
* feat(discover): Show more relevant shows in upcoming TV
* chore(release): 🚀 v4.20.0
* fix(sync): Emby+Jellyfin - sync multi-episode files of 3+ episodes
* perf(sync): Emby+Jellyfin - use a more reliable filter to missing items
* fix(sync): Emby+Jellyfin - sync multi-episode files of 3+ episodes [skip ci]
* fix: added media type tag to media type text (#4638)
[skip ci]
* fix(sickrage): Fixed issue with incorrect handling of SiCKRAGE episode results returned during episode status changes, now expects array of objects from data path if present (#4648)
[skip ci]
* fix: Missing Poster broken link fix (#4637)
[skip ci]
* 🌐 Translations Update (#4622)
[skip ci]
* Update launch.json (#4650)
[skip ci]
* fix: Improve Swagger documentation (#4652)
* Upgrade Swashbuckle dependency
* Document /token response
* Add support for Newtonsoft annotations in Swagger
* Remove unecessary ActionResult [skip ci]
* fix(API): Fix pagination in some edge cases (#4649)
[skip ci]
* 🌐 Translations Update (#4655)
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(discover): Carousel touch not working when scrolling page and recommendations and similar movie navigation (#4633)
* fixed touch not working on carousels
* fixed touch not working
* Movie details component fixes
Fixed recommendations and similar not changing the data on the component by calling the init function again on param change
Moved the ngif results > 0 to the mat-expansion panel to avoid rendering the entire element if it doesn't have any results instead of having an empty panel.
* removed unused line, added scroll to top on init
* updated recommendation refresh implementation
Changed the implementation to use the router instead in order to reload the component instead of just reloading the data.
This implementation makes sure the component gets destroyed on navigation eliminating any memory leaks, reloading CSS in case of having animations on page load and generally a continuation of the experience you get when you browse into a movie from the discover page.
* chore: 👥 Updated Contributors [skip ci]
* chore(release): 🚀 v4.20.1 [skip ci]
* fix: 🐛 Fixed the Request on Behalf of having blanks (#4667)
* chore(release): 🚀 v4.20.2 [skip ci]
* fix(plex): 🐛 Fixed an issue with the Plex Sync
* chore(release): 🚀 v4.20.3 [skip ci]
* fix (technical): Improved some of the date time parsing handling
* fix: fixed build
* chore(release): 🚀 v4.20.4 [skip ci]
* feat: Upgrade to Angular14 (#4668)
* refactor: 🔥 removed angular-bootstrap-md dependancy
* chore: update tsconfig
* yeah
* ng14 upgrade
* refactor: migration changes
* fix: fixed CLI
* test: Fixed automation
* chore: 👥 Updated Contributors [skip ci]
* perf: stop populating obsolete subscribe fields (#4625)
* chore(release): 🚀 v4.21.0 [skip ci]
* fix(images): Retry images with a backoff when we get a Too Many requests from TheMovieDb #4685
* chore(release): 🚀 v4.21.1 [skip ci]
* 🌐 Translations Update (#4683)
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix: Landing and Login page improvements (#4690)
* chore(release): 🚀 v4.21.2 [skip ci]
* feat(discover): ✨ Added infinite scroll on advanced search results
* feat(discover): ✨ Added infinite scroll on advanced search results
* chore(release): 🚀 v4.22.0 [skip ci]
* 🌐 Translations Update (#4694)
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(discover): 🐛 Created new Image component to handle 429's from TMDB (#4698) and fixed#4635 (#4699)
* chore(release): 🚀 v4.22.1 [skip ci]
* fix: fixed an issue where I broke images for some users
* chore(release): 🚀 v4.22.2 [skip ci]
* ci(Mergify): configuration update (#4701)
Signed-off-by: Jamie <tidusjar@gmail.com> [skip ci]
* fix: Override Sonarr V3 Profiles endpoint (#4678)
* Override Sonarr V3 Profiles endpoint [skip ci]
* fix(4K) :4K request fixes (#4702)
* GetRequestsByStatus wasn't implementing the MovieRequests object correctly for 4K quality requests with the ProcessingRequest status.
* Fixed 4K requests not getting automatically approved if the user has the "Auto Approve Movie" role flag enabled.
* Fixed "Request Date" values for the "left-panel-details" div class. Previously when the movie was exclusively 4K (regular request was absent), then "Request Date" equaled DateTime.MinValue (January 1, 0001).
* Fixed "Request Status" evaluation in the "left-panel-details" div class. Now it shows the appropriate status instead of an empty spot. "Request Status" displays both regular and 4K statuses at the same time if needed. Added a comma to the end of the "RequestStatus" label to maintain design consistency with the other labels. Also added a "Denied Reason" element for 4K requests.
* chore: 👥 Updated Contributors [skip ci]
* chore(release): 🚀 v4.22.3 [skip ci]
* chore: Storybook (#4700)
[skip ci]
* chore: Translations
[skip ci]
* 🌐 Translations Update (#4704)
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci] (#4713)
* build: Run automation tests in docker (#4715)
[skip ci]
* fix: fixed trakt image not loading when base url present (#4711)
[skip ci]
* fix: 🐛 Fixed missing externals (#4712)
* chore: 👥 Updated Contributors [skip ci]
* chore(release): 🚀 v4.22.4 [skip ci]
* test: fixed automationt tests [skip ci]
Co-authored-by: sephrat <34862846+sephrat@users.noreply.github.com>
Co-authored-by: Conventional Changelog Action <conventional.changelog.action@github.com>
Co-authored-by: Teifun2 <Teifun2@users.noreply.github.com>
Co-authored-by: contrib-readme-bot <contrib-readme-action@noreply.com>
Co-authored-by: dr3amer <91037083+dr3am37@users.noreply.github.com>
Co-authored-by: echel0n <echel0n@sickrage.ca>
Co-authored-by: Marley <55280588+marleypowell@users.noreply.github.com>
Co-authored-by: Igor Borges <igor@borges.dev>
Co-authored-by: Lucane <Lucane@users.noreply.github.com>
Co-authored-by: mkgeeky <github@mkgeeky.xyz>
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* GetRequestsByStatus wasn't implementing the MovieRequests object correctly for 4K quality requests with the ProcessingRequest status.
* Fixed 4K requests not getting automatically approved if the user has the "Auto Approve Movie" role flag enabled.
* Fixed "Request Date" values for the "left-panel-details" div class. Previously when the movie was exclusively 4K (regular request was absent), then "Request Date" equaled DateTime.MinValue (January 1, 0001).
* Fixed "Request Status" evaluation in the "left-panel-details" div class. Now it shows the appropriate status instead of an empty spot. "Request Status" displays both regular and 4K statuses at the same time if needed. Added a comma to the end of the "RequestStatus" label to maintain design consistency with the other labels. Also added a "Denied Reason" element for 4K requests.
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
* fix(translations): 🌐 New translations from Crowdin [skip ci]
