search

BESTRobotics / registry

A national registry for information pertaining to BESTRobotics
0 stars 0 forks source link

New user email activation #45

Closed gdyoung93 closed 5 years ago

gdyoung93 commented 5 years ago

Need to add proper link for account activation. The link in the email is using localhost and does not work.

gdyoung93 commented 5 years ago

Apparently it is not needed. I was able to create account and login without activation through email link.

the-maldridge commented 5 years ago

As mentioned in our earlier email, activation requirements are currently disabled because the system is configured with the following directive:

[dev]
activate = true

In this mode the check in https://github.com/BESTRobotics/registry/blob/master/internal/http/accounts.go#L58 will suppress the activation initializer and create accounts that are fully activated without clicking through the email. The email is still sent though for correctness reasons.

As for the link having the wrong host I have corrected the wrong host and reloaded the configuration, new emails appear to have the correct host.