eisbuk / EisBuk

Manage bookings for ice skating lessons
GNU Affero General Public License v3.0
0 stars 1 forks source link

Feature/single account multiple athletes #917

Closed ikusteu closed 7 months ago

ikusteu commented 7 months ago

Fixes #904

github-actions[bot] commented 7 months ago

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

github-actions[bot] commented 7 months ago

Conflicts have been resolved. A maintainer will take a look shortly.

github-actions[bot] commented 7 months ago

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

ikusteu commented 7 months ago

Looks great in general. Needs a fix though.

I tested this in production (AFAIU there are no changes to cloud functions in this PR, so it's safe to test it through the preview channel deployed by the CI) and I was able to:

  • register a new athlete
  • see the newly registered athlete amongst the choices in /select_account

I was not able to:

  • see my other, already existing, account with the same email
  • see my other, already existing, account with the same phone number

I've used eisbuk.web.app (the preview channel) to test this out: I've created a new athlete (with my existing account). I was able to see both and navigate between...

Did you (maybe) mean: You already had two accounts with the same email and expected to see them both in the selection (now with the multiple athlete update)...if so, yeah, that should be working, but if it's not, maybe we should debug that together...

github-actions[bot] commented 7 months ago

Conflicts have been resolved. A maintainer will take a look shortly.

github-actions[bot] commented 7 months ago

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

github-actions[bot] commented 7 months ago

Conflicts have been resolved. A maintainer will take a look shortly.

github-actions[bot] commented 7 months ago

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

github-actions[bot] commented 7 months ago

Conflicts have been resolved. A maintainer will take a look shortly.