VerifyEmail should not be called on the server.

meteor-useraccounts / flow-routing

Useraccounts packages add-on for integration with Flow Router and Blaze Layout.

https://atmospherejs.com/useraccounts/flow-routing

MIT License

72 stars 50 forks source link

VerifyEmail should not be called on the server. #26

Closed timothyarmes closed 8 years ago

timothyarmes commented 8 years ago

When using FlowRouter 3 with SSR support the routes are also executed on the server, however Accounts.verifyEmail() (which is executed when visiting the atVerifyEmail route) is only available on the client and caused a server error. This PR fixes that.

splendido commented 8 years ago

Tnx! ...could you also have a look at the resetEmail and enrollAccount routes? I don't expect calls to undefined methods but I'm curious to see whether the token is correctly picked up...

timothyarmes commented 8 years ago

I can confirm that the reset password link works fine for me. I can't easily test the enrolment email on my current project.

splendido commented 8 years ago

Ok, fine, they share the same logic... Tnx!