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

Hide subscribe and unsubscribe buttons for own changesets #827

Closed
scaidermern opened this issue Nov 6, 2014 · 13 comments
Closed

Hide subscribe and unsubscribe buttons for own changesets #827

scaidermern opened this issue Nov 6, 2014 · 13 comments

Comments

@scaidermern
Copy link

Currently there is a subscribe button for your own changesets although you are automatically subscribed to them. After someone commented on your own changeset this button gets replaced by an unsubscribe button.

In my opinion there should be neither a subscribe nor an unsubscribe button for your own changesets as the changeset author is automatically subscribed and should stay subscribed.

There has been a related question on help.

@tomhughes
Copy link
Member

I'm not sure we have any right to force a changeset author to receive email about their changeset? I can even see an argument that it would break data protection law to do so...

@scaidermern
Copy link
Author

I see but in that case there should be an option for disabling changeset comment notifications completely. And there is still the bug of the existing subscribe button for own, not yet commented changesets to which authors are automatically subscribed.

@tomhughes
Copy link
Member

Well there shouldn't be a subscribe button if you are already subscribed, there should be an unsubscribe button...

Are you sure you aren't looking at an old changeset from before comments existed, to which you won't be subscribed?

@scaidermern
Copy link
Author

That's a good question. When looking at my own new changesets there is only an unsubscribe button. But the author of the question at help did see only a subscribe button, yet he received a mail about me commenting his changeset. Maybe he looked at the wrong changeset and there is no bug at all.

@tomhughes
Copy link
Member

There is nothing stopping you commenting on old changesets, but the author won't be subscribed so won't get emailed (this is issue #823).

@jase88
Copy link

jase88 commented Nov 6, 2014

I was the one asking on help.osm.org
Yeah @tomhughes You're right it probably was an old changeset that I was viewing. I didn't know you're not automatically subscribing to the "old" changesets (wasn't mentioned in a blog post or somewhere public - except github). So it is pretty confusing to have a subscribe or unsubscribe button on changesets (of course depending on the date). So I guess issue #823 should be fixed.

I'm sure I hit the unsubscribe on the one changeset and received an email notification. Tried to reproduce it but had no luck. I guess there is no log or history to verify the status of subscription that, right? Just got that mails...

@tomhughes
Copy link
Member

So you're saying you got an email after you had unsubscribed? If you give me the number of the changeset then I can look into that.

@jase88
Copy link

jase88 commented Nov 6, 2014

Hi,
scai has left a comment on one of your changesets created at 2014-11-06 11:38:07 UTC without comment

A new comment.

More details about the changeset can be found at http://www.openstreetmap.org/changeset/26581590.

Hopefully the error wasn't on my side and I misclicked the button or something. Thanks

@tomhughes
Copy link
Member

Well you're definitely not subscribed to that changeset currently...

@tomhughes
Copy link
Member

It looks like you unsubscribed twice, at 11:35:49 and again at 14:09:35 but that shouldn't be possible...

Can you explain exactly what sequence of actions was taken after this changeset was created?

@jase88
Copy link

jase88 commented Nov 6, 2014

Okay, I will try to remember:

  1. changeset created
  2. scai commented -> got email
  3. wrote own comment
  4. clicked unsubscribe
  5. wrote own comment
  6. scai commented -> email again
  7. unsubscribed again

@tomhughes
Copy link
Member

I can only think that the first unsubscribe failed somehow, but I can't find any sign of a failure in the logs...

@tomhughes
Copy link
Member

OK So the answer was staring me in the face all along...

The "problem" is that when you wrote a comment at step 5 that resubscribed you - anybody that comments on a changeset is subscribed to it if they aren't already subscribed. So I think this is all working as designed and there is no problem here.

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

No branches or pull requests

3 participants