Skip to content
This repository has been archived by the owner on Sep 29, 2022. It is now read-only.

feat: handle cases where there are no customer number set on profile #478

Merged
merged 2 commits into from
Feb 10, 2022

Conversation

mikaelbr
Copy link
Contributor

Jf. tidligere feature rundt visning av customer number (#469), gjorde jeg på en måte en liten blunder men som egentlig er en fordel: Jeg tok ikke høyde for at customer number ikke kunne eksistere. Dette for at vi egentlig skal kunne ha customer number uansett ved innlogging. Men jeg glemte å ta høyde for tilfeller hvor registrering av bruker har feilet.

Historisk sett har skjedd litt pga ymse årsaker og det har vært vanskelig å fange opp. Det er egentlig ikke noe som skal skje, men det kan skje en gang i jubelåret og da ønsker vi at kunden tar kontakt. Forslaget mitt er en enkel melding som viser det her:

Screenshot 2022-02-10 at 08 34 04

Vi kan velge å gjøre mer ut av det med større varsel osv, men dette er egentlig en edge case (som har vist seg å ha skjedd noen ganger).

@github-actions
Copy link

github-actions bot commented Feb 10, 2022

Visit the preview URL for this PR (updated for commit 6c18f50):

https://atb-webshop-staging--pr478-mikael-msg-customern-fxz7jflq.web.app

(expires Sat, 12 Feb 2022 07:45:20 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@mikaelbr mikaelbr merged commit 217e5d5 into master Feb 10, 2022
@mikaelbr mikaelbr deleted the mikael/msg-customerNumber branch February 10, 2022 10:12
@tormoseng
Copy link
Member

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants