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

Remove all verification tokens for a specific email once that email is verified #4626

Closed
zimme opened this Issue Jun 25, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@zimme
Contributor

zimme commented Jun 25, 2015

This will break links in the "other" verify email emails, but it will keep the user record cleaner

@zimme

This comment has been minimized.

Contributor

zimme commented Jun 25, 2015

I'll try and put together a PR for this unless you think this is a bad idea

@stubailo

This comment has been minimized.

Contributor

stubailo commented Jun 30, 2015

Yeah, I think this would be a good idea to do.

Thanks for the feature request! We welcome discussions about how to make Meteor better. If you haven't yet, check out our wiki page about feature requests on GitHub.

If you add a PR for this I'll take a look.

@zimme

This comment has been minimized.

Contributor

zimme commented Jun 30, 2015

zimme added a commit to zimme/meteor that referenced this issue Jul 1, 2015

Remove all verificationTokens for a specific email once verified
This will break the other verification email links but it will keep the
user document cleaner. The email will be verified anyways so who cares
that the other verification links isn’t working any more.

Closes meteor#4626

@stubailo stubailo closed this in f91da45 Jul 7, 2015

zimme added a commit to Favro/meteor that referenced this issue Jul 28, 2015

Remove all verificationTokens for a specific email once verified
This will break the other verification email links but it will keep the
user document cleaner. The email will be verified anyways so who cares
that the other verification links isn’t working any more.

Closes meteor#4626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment