Option to change the users mail adress about password-manager

[zeruniverse]

This is easy to implement. But it's not safe and I don't think people will change email address. You can modify your email in database anyway

[nickbe]

Not safe? Why?

[zeruniverse]

Sorry. In the version I use, I integrated email verification.

I just took a look at the version on Github. Seems we don't need email at
all. That's a redundant.

On Thursday, 16 June 2016, Nick [email protected] wrote:

Not safe? Why?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#56 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEbvNDnXfBWlzH0Id2VBzBAhkufLdlDTks5qMfebgaJpZM4I3Odo
.

[nickbe]

So if we could simply enter mail adresses as user names?
Is it possible to change the username? Does that lead to problems?

[zeruniverse]

I'll consider remove the email feature. It isn't used anywhere.

On Friday, 17 June 2016, Nick [email protected] wrote:

So if we could simply enter mail adresses as user names?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#56 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AEbvNBKEAdNcPiOowZfFFyhlJzwCSRd4ks5qMl0MgaJpZM4I3Odo
.

[zeruniverse]

Ok I took a look at the project. The email field was introduced in the first commit 4ea00d2. And I noticed the first version has the feature of email verification so it's necessary to have an email field. But later I removed this feature as I noticed the implementation of sending emails might vary and not all people want this. Meanwhile I put it in README as possible extensions.

Yes, I can confirm for current version, the email field does nothing. But I think it's better to keep it because then adding email verification feature doesn't involve in database change. As for the option to change email address, it seems this is only useful if email does something while it actually doesn't. So I won't implement it.