Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

can't get several command_aliases to work #79

Closed
hhenne1 opened this issue May 16, 2023 · 3 comments
Closed

can't get several command_aliases to work #79

hhenne1 opened this issue May 16, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@hhenne1
Copy link

hhenne1 commented May 16, 2023

Hi,
i love your bot, but i'm having an issue setting it up.
no matter what i do or try, i cant get sonarr_series_command_aliases and readarr_book_command_aliases to work with several values.
as soon as i put several values for sonarr, the docker container (toddrob/searcharr:latest) just loops with a python error.
setting several values for radarr works fine.
and setting any value for readarr_book_command_aliases never works, not even with a single one, and reverts to its default.

2023-05-16 18:58:50,731 - INFO - searcharr(139795833956160):66 - Searcharr v3.2.1 - Logging started!
2023-05-16 18:58:51,895 - WARNING - searcharr(139795833956160):362 - No readarr_book_command_aliases setting found. Please add readarr_book_command_aliases to settings.py (e.g. readarr_book_command_aliases=["book", "bk"]. Defaulting to ["book"].

i attached the settings.py with the lines i tried.

settings.py.txt

maybe someone has a hint what i'm doing wrong?

@hhenne1
Copy link
Author

hhenne1 commented May 16, 2023

wops.. had an alias with a space in sonarr_series_command_aliases .. that fixed the reboot-loop..
readarr_book_command_aliases still does not get recognized though..

@hhenne1
Copy link
Author

hhenne1 commented May 16, 2023

sorry that i keep adding notes to this, but.. if i just use readarr_book_command_aliases, it does not get recognized and defaults to book, but runs.
if i replace it with readarr_movie_command_aliases, the app crashes and bootloops.
but if i use both, it works just the way i hoped it would.

the configuration-help (https://github.com/toddrob99/searcharr/wiki/Configuration-::-settings.py) states under readarr_book_command_aliases -> Example: readarr_movie_command_aliases = ["book", "b"]

thats why i thought it would be worth a shot..
so, i got it working now, though it looks quite weird in the config.
thanks :*

@hhenne1 hhenne1 closed this as completed May 16, 2023
@toddrob99 toddrob99 reopened this May 17, 2023
@toddrob99 toddrob99 self-assigned this May 17, 2023
@toddrob99 toddrob99 added the bug Something isn't working label May 17, 2023
@toddrob99
Copy link
Owner

Thanks for posting. There is actually a bug in the searcharr code that it's looking for readarr_movie_command_aliases instead of readarr_book_command_aliases, and if the movie version isn't in the settings file, it will set the book version (the actual setting) to the default value. I'm sure this was a copy/paste mistake. I will be fixing this in v3.2.2 shortly. I also corrected the wiki.

https://github.com/toddrob99/searcharr/blame/main/searcharr.py#L360

@toddrob99 toddrob99 mentioned this issue May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants