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

Show option "Unsubscribe from..." instead of "Subscribe to..." if already subscribed #49

Open
imsodin opened this issue Jun 29, 2016 · 5 comments
Labels

Comments

@imsodin
Copy link

imsodin commented Jun 29, 2016

Under "Subscriptions" where all the channels I subscribed to are listed everything works as expected: Opening the context menu of one such channel I get the option to "Unsubscribe". However under "My Subscriptions" where new videos of my subscribed channels are listed, the option "Subscribe to..." is available even though I have already subscribe to this channel (same is true under "Recommendations", not tested in other menues). I would expect to have an option "Unsubscribe from..." if I am already subscribed.

@h0d3nt3uf3l
Copy link
Collaborator

Hey,
is the function to unsubscripe valid? I mean, If you click at subscribe to when you allready done this, can you unsubscripe the channel?

@h0d3nt3uf3l h0d3nt3uf3l added the bug label Jul 9, 2016
@imsodin
Copy link
Author

imsodin commented Jul 11, 2016

Subscribing to a channel works as expected. If I active "Subscribe to ..." while already being subscribed to a channel this has no effect, there is no notification message and I stay subscribed to the channel. If I choose "Unsubscribe" in the menu "Subscriptions", I successfully unsubscribe from that channel (however it keeps being displayed in "Subscriptions" until you leave this menu and reenter, which is no big issue).
So I guess the addon needed to somehow test whether the channel belonging to the video in question is already subscribed when opening the context menu or a bit later when choosing a possible new option "(Un-)Subscribe" and then invoke the correct action unsubscribe or subscribe, which are already correctly implemented.

@jdf76
Copy link
Collaborator

jdf76 commented Nov 8, 2016

I hate to ask but if this still an issue and can you run it by me one more time what the issue is.

I think you're saying you can't unsubscribe from a channel, but i'm not really sure where or when you're having this issue.

@disrupted
Copy link
Contributor

what I think this is about: in the "my subscriptions" folder you should either have the possibility to unsubscribe from channels (in the context menu it currently always only says "subscribe to") or hide this context menu entry from that page because you're already subscribed to these channels.

@imsodin
Copy link
Author

imsodin commented Nov 12, 2016

freeman212 is quite correct. And the last suggestions is probably the easiest to implement: Just remove the menu entry "Subscribe to..." when one is in "My Subscriptions" as one is obviously already subscribed.
I guess the problem is that when opening the context menu on a youtube video (not channel) it is not checked whether I have already subscribed to the channel, which has published that video. Ideally such a check would be done and I would be presented with either "Subscribe to..." if I am not yet subscribed or "Unsubscribe from..." if I already am.
However this is not a breaking issue. The UI is not consistent in the current state, but there are ways to subscribe to and unsubscribe from channels - so don't worry if resources aren't here to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants