RFC: Alias search #875
Open
RFC: Alias search #875
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
|
@ta264 do we need to pull the fix commit from this into 0.7? |
|
Yeah we probably do. I was also wondering about pulling in the change to pull aliases into the dB at least for parsing, even if we save the alias search for 0.8 |
|
Actually we pulled the fix commit across in #879 |
bakerboy448
added
conflict
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.
Database Migration
NO
Description
Hooks up artist / album alias for indexer search and result parsing. So far only in place for newznab.
Currently it looks for a "best" query by looking through titles/aliases for one that contains no unicode or special characters. If one isn't found, it takes the main title. Results are fetched in tiers until we get a result (so if the main query returns results it doesn't search with any aliases).
Todos
Issues Fixed or Closed by this PR