Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Conversation

@brianlovin
Copy link
Contributor

Status

  • WIP
  • Ready for review
  • Needs testing

Deploy after merge (delete what needn't be deployed)

  • api

Should close #4686

Copy link
Contributor

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well that was simple enough! LGTM!!

@brianlovin
Copy link
Contributor Author

Honestly have no clue how this wasn't caught until now, I guess most people don't want @mention notifs turned off :P

@brianlovin
Copy link
Contributor Author

Mmm flaky e2e test failures 😭

@brianlovin brianlovin merged commit 8c2a197 into alpha Feb 20, 2019
@brianlovin brianlovin deleted the fix-mention-unsub-link branch February 20, 2019 19:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unsubscribe link in new mention notification emails does not work

3 participants