topcoder-platform / tc-notifications

4 stars 15 forks source link

Unsubscribe link #63

Closed gondzo closed 6 years ago

gondzo commented 6 years ago

In the current emails we have two links to unsubscribe - one at sendgrid and one at connect image

@vikasrohit can we remove the sendgrid link? I'm not sure where in sendgrid is it configured

vikasrohit commented 6 years ago

@gondzo Its difficult to remove the sendgrid one because that is present in the compulsory unsubscribe section of emails. If we remove that, we have to add the unsubscribe option to each and every other template where we require unsubscribe option.

I think we are okay with these two options because Notification Settings lets the user choose the specific event they want to op in or opt out while the default sendgrid option lets them disable any communication from topcoder.

vikasrohit commented 6 years ago

@vic-topcoder @acshields your thoughts?

gondzo commented 6 years ago

The main issue I have with that unsubscribe link is that there is no way to undo that if I click it accidentally. Maybe we could also add sendgrid subscription preferences link too https://sendgrid.com/docs/ui/sending-email/recipient-subscription-preferences/ (I'm not sure if this will let the user subscribe again after he unsubscribed, but it sounds like it)

vikasrohit commented 6 years ago

I guess that is product call. IMO, the global unsubscribe link is more or less for legal compliance while our Notification Settings allows users to unsubscribe/subscribe from specific group of emails. Now, if we use Sendgrid's email preferences, that might cause confusion to the users and we might have to provide an integration between Sendgrid and our email preferences. So, I would rather not like to go with that at present.

fyi @vic-topcoder @acshields

acshields commented 6 years ago

@vikasrohit - agreed with your assessment.

vikasrohit commented 6 years ago

Thanks @acshields for the confirmation. Closing the issue @gondzo.