loomchild / sojourner-web

FOSDEM conference companion PWA
https://sojourner.rocks
GNU Affero General Public License v3.0
50 stars 6 forks source link

Error that email is in use, but that's not possible #126

Closed thibaultmol closed 1 year ago

thibaultmol commented 1 year ago

Hi, I just got this error while trying to register on the site Screenshot 2023-02-01 16 39 46 Even though the email I signed up with is unique (I just created it using SimpleLogin). So I'm guessing it's more like a ban on using .link TLD's?

loomchild commented 1 year ago

Hi, thanks for reporting.

Yes, it's possibly a ban. I am simply using Firebase auth, so they might be blocking some domains. Weird that the error message is this, but again, I am simply displaying it.

loomchild commented 1 year ago

Actually, I was able to register to Sojourner using email generated via SimpleLogin (but not on .link TLD). Can you try another email, perhaps it's really taken? Or perhaps there was an error during first registration and the account was finally created?

thibaultmol commented 1 year ago

Ok so seems like it did actually make the account, it just threw that error.

Actually actually: maybe I clicked the register button twice or something. I can't recreate it.

I would like to note though that this implementation of the material design notifications at the bottom; have the problem that newer notifications don't go over it. Like if I logout accidentally and then log back in. the login message won't appear until the 'successful logout' notification has gone away. I think it might be better if it were to show the 'successful login' above the 'logout'. This is an example. But I meant that newer notifications should display over old notifications

also: I understand the icon change for logged in/out. but it's kinda confusing. maybe an actual logout button would be more clear?

loomchild commented 1 year ago

These are valid suggestions, could you please create separate tickets for them (I am afraid I won't be able to work on them before Saturday), and I will close this ticket.