search

TheWicklowWolf / Lidify

Music discovery tool that provides recommendations based on selected Lidarr artists, using Spotify or LastFM.
MIT License
152 stars 0 forks source link

False message Artist is already in Lidarr. #7

Closed bryceprutsos closed 8 months ago

bryceprutsos commented 8 months ago

Hey just came by to show the app some love.

I noticed that when I have everything setup and select an artist on the webfront it says Artist is already in Lidarr. However looking through the logs it shows this

lidify | Artist 'Metro Boomin' matched 'Metro Boomin' with MBID: 59db3d82-86ea-451f-881f-dffc8ec387c9 Ratio: 100 lidify | Failed to add artist 'Metro Boomin' to Lidarr. lidify | {"propertyName": "Path", "errorMessage": "Invalid Path: '/data/media/music/Metro Boomin/'", "attemptedValue": "/data/media/music/Metro Boomin/", "severity": "error", "errorCode": "PathValidator", "formattedMessageArguments": [], "formattedMessagePlaceholderValues": {"path": "/data/media/music/Metro Boomin/", "propertyName": "Path", "propertyValue": "/data/media/music/Metro Boomin/"}} lidify | Artist 'Metro Boomin' is already in Lidarr.

I know its a problem with my Lidarr setup but figured you should know so you can correct the error message back to the user.

bryceprutsos commented 8 months ago

Also maybe it could just grab the root folder from Lidarr and can it start searching on add?

TheWicklowWolf commented 8 months ago

Also maybe it could just grab the root folder from Lidarr and can it start searching on add?

In certain scenarios, multiple root folders may exist, necessitating the need for an option to select the desired one.

Thanks for your input.