-
Notifications
You must be signed in to change notification settings - Fork 422
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
unsubscribe link from email not working #662
Comments
Hi @pdcoutinho. Thanks for submitting this issue. Could you please submit a patch to fix it? That would be great. Changing the route to be a |
Rather than an authenticity token, could we instead have a unique hash on the topic + subscription that is generated + then is passed through in this request? The route will change to a I'm thinking something such as |
Patches welcome to fix this :) I do not have time myself. |
Turns out I do have time to fix this. Please review #669 and let me know if it's suitable for you. |
Users are sent an email if their subscribed topics are replied.
In said email, there is a link to unsubscribe. That link issues a GET request, but routes show a POST, and the website breaks.
http://example.com/forums/some-forum/topics/some-topic/unsubscribe
forem (1.0.0.beta1)
Rails 4.2.0
no route matches get unsubscribe bla bla bla bla ^^
The text was updated successfully, but these errors were encountered: