The verify_addr method redirects to "/"; it'd be more useful to go to "/auth/preferences"
On branch db/7523 (and also some SF internal changes on forge-classic:db/7523)
I also added a flash notification upon submitting an email address, and a config option to prevent deleting your last verified address.
You still can delete last verified email if it is not primary.
Hm, checking code one more time I see that a new primary should be set when you delete the old one, but it does not work for me somehow...
08:12:25,863 WARNI [forgesf.sf_providers] "None" was given for a preferred email address for admin1, selecting email@example.com instead
but firstname.lastname@example.org is the address I've deleted. Seems like select_new_primary_addrin sf provider returns deleted address somehow.
It duplicates only when your primary address is the first address in the addresses list. Maybe add a test for this case along with the fix?
You need user_prefs_storage.method = sf in production.ini to reproduce.Local storage method works fine.
user_prefs_storage.method = sf
Fixed on forge-classic:db/7523 for deleting primary addr when it is listed first, with a test
And rebased both branches to get some recent adjacent verification mail changes incorporated and test updated.
Log in to post a comment.