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 optional requirements, create new Settings tab #548

Closed
dagwieers opened this issue Oct 25, 2019 · 0 comments · Fixed by #552
Closed

Remove optional requirements, create new Settings tab #548

dagwieers opened this issue Oct 25, 2019 · 0 comments · Fixed by #552
Labels
enhancement New feature or request question Further information is requested
Milestone
v2.3.0

Comments

@dagwieers
Copy link
Collaborator

dagwieers commented Oct 25, 2019 •
edited
Loading

Describe the feature

It is probably better to remove all optional requirements from the add-on, and add specific entries in the settings to install, enable and configure these optional add-ons.

Not only is it more user-friendly, it offers a cleaner integration using a single user-interface.
Probably have a new Settings tab:

Integration

  • Install YouTube add-on (when not already installed)
  • Enable YouTube integration (when installed)
    • YouTube settings...
  • Install Up Next add-on (when not already installed)
  • Enable Up Next integration (when installed)
    • Up Next settings...
  • Install PySocks addon (when not already installed)

Since yesterday's problems at VRT NU, Iit could also be useful to have Twitter integration to show the most recent VRT NU twitter replies. It would have confirmed VRT NU issues almost immediately. How to do such integration is something yet to be discovered ;-) See #551
@dagwieers dagwieers added enhancement New feature or request question Further information is requested labels Oct 25, 2019
@dagwieers dagwieers added this to the v2.3.0 milestone Oct 25, 2019
@dagwieers dagwieers mentioned this issue Oct 26, 2019
Merged
@dagwieers dagwieers mentioned this issue Jan 20, 2020
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Milestone
v2.3.0
Development

Successfully merging a pull request may close this issue.

Implement Integration settings dagwieers/plugin.video.vrt.nu
1 participant
@dagwieers