search

beyond-all-reason / bar-lobby

BAR Lobby Client
https://beyond-all-reason.github.io/bar-lobby/
MIT License
33 stars 30 forks source link

Update LoginForm.vue to include case sensitivity note #208

Closed Penguin2600 closed 1 year ago

Penguin2600 commented 1 year ago

I had trouble logging in and opened a ticket and wasted some poor fool's time because I didn't expect Email to be a case sensitive field.

This change updates the email box to have a note about case sensitivity, which will hopefully reduce the support workload for staff.

WatchTheFort commented 1 year ago

Why is email case sensitive at all? @Jazcash will anything break if it's made case insensitive?

Penguin2600 commented 1 year ago

Why is email case sensitive at all? @Jazcash will anything break if it's made case insensitive?

I agree making email case-insensitive would be more intuitive IMO. Happy to try and submit a PR for that if we decide that is a better direction. (In the support ticket I was told that case-sensitivity was expected behavior)

Jazcash commented 1 year ago

Why is email case sensitive at all? @Jazcash will anything break if it's made case insensitive?

Not sure, but it's a server problem, not the lobby's.

Penguin2600 commented 1 year ago

Turns out this is the wrong codebase. Thanks everyone for Pointing me in the right direction, I'll Open a new PR in the correct place.