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.

Sign up for GitHub

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

Jump to bottom

Remove Plex login requirement #61

Closed
Ephellon opened this issue Dec 13, 2018 · 5 comments
Closed

Remove Plex login requirement #61

Ephellon opened this issue Dec 13, 2018 · 5 comments
Assignees
@Ephellon
Labels
enhancement this item makes the project better in some way help-wanted outside assistance is recommended for this item

Comments

@Ephellon
Copy link
Collaborator

Ephellon commented Dec 13, 2018 •
edited
Loading

Suggested by @bolhaskutya in #56:

I only want to use the addon to send to radarr

This would require a slight overhaul, as the code is designed to search Plex for matching media... Maybe a dummy machineID can be used, as long as they have a Plex account?
@Ephellon Ephellon added enhancement this item makes the project better in some way help-wanted outside assistance is recommended for this item labels Dec 13, 2018
@Ephellon Ephellon self-assigned this Dec 13, 2018
@defaultsecurity
Copy link

Ahh, I see. So you actually check for movies using plex, not just simly sending them to radarr/sonarr. I didn't realize that.

@Ephellon
Copy link
Collaborator Author

Yes. Sorry about the confusion

@Ephellon Ephellon mentioned this issue Dec 15, 2018
Closed
@Ephellon
Copy link
Collaborator Author

Note to self: make a DO_NOT_USE variable, then it's extremely easy to tell the extension not to use Plex

@Ephellon Ephellon added in-progress this item is being worked on and removed in-progress this item is being worked on labels Dec 16, 2018
@Ephellon
Copy link
Collaborator Author

Done. It will cause errors for users not paying attention.

I added prompts to try and tell/deter the user from not using a server.

@Ephellon Ephellon mentioned this issue Dec 19, 2018
Merged
@defaultsecurity
Copy link

Awesome!

This was referenced Jun 28, 2019
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ephellon Ephellon

Labels
enhancement this item makes the project better in some way help-wanted outside assistance is recommended for this item
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Ephellon @defaultsecurity