Skip to content

Add concrete season selection, narrow rootFolder names, notify admin in different chat and add requester user tag to sonarr/radarr#143

Merged
Waterboy1602 merged 8 commits intoWaterboy1602:masterfrom
anast20sm:master
Dec 20, 2023
Merged

Conversation

@anast20sm
Copy link
Contributor

I add some functions that are useful in my case, where each user can add series/movies on their own chat, I track in a separated channel what is done and I add the tags in order to configure sonarr/radarr/tautulli notifications.

  • Select multiple seasons and then send
  • Narrow root folder names to the folder name instead of full path (in my case some of them are too long). Can be enabled in config
  • Notify admin into a separate channel/group/chat by providing id in the config
  • Add tag from who requests the movie/series. Tag must be already created in sonarr/radarr (#TODO create if new user) (in my case i use this to configure notifications for each user in sonarr/radarr)

I think these issues can be closed:
#116
#113
Not #84 but the notifications added could help if this is the requirement

PS: I do not have docker build knowledge by now, can someone build the image for me please ? :)

@Waterboy1602
Copy link
Owner

Thank you for the pull request and the additions that you made! These are a lot of improvements.

What do you think about making the selection of the seasons a bit more fluent by adding a button to directly select all the current and future seasons with one press?

I was also thinking about adding some explanation about the new settings in the config_example.yaml-file. I don't think the new settings are so self-explanatory.

I've tried building the new Docker image and it builds without problem.

@anast20sm
Copy link
Contributor Author

anast20sm commented Nov 26, 2023

Just added what you commented @Waterboy1602 ! Many thanks for the review and feedback!

@Waterboy1602
Copy link
Owner

Everything now looks alright for me. Thank you for the contributions and the pull request!

After the merge I will close issue: #113 & #116

@Waterboy1602 Waterboy1602 merged commit 99bdf75 into Waterboy1602:master Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants