Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add customer number to Profile #2118

Merged
merged 2 commits into from
Feb 7, 2022
Merged

Conversation

cbrevik
Copy link
Contributor

@cbrevik cbrevik commented Feb 3, 2022

Related AtB-AS/webshop#469

Det store spørsmålet er, skal kundenummer vises om du ikke er logget inn med telefonnummer? Du er anonym, men du har fortsatt et kundenummer (for at du skal kunne gjennomføre et kjøp). Men gir det mening å vise det? Jeg synes på sett og vis det, siden det gjør det mulig for kundeservice å hjelpe de selv om de er anonym

image

@mikaelbr
Copy link
Collaborator

mikaelbr commented Feb 3, 2022

Uten å ha tenkt alt for mye på det så tenker jeg at det er minst like verdifullt i anonyme kjøp. Og jeg tror ikke det ødelegger noe for anonymiteten?

@cbrevik
Copy link
Contributor Author

cbrevik commented Feb 3, 2022

Nei det kan ikke knyttes opp mot en gitt person, annet enn hvis de velger å dele det. Så jeg synes det er helt innafor også

@jahnrohme
Copy link

Enig med resonnementet deres.
Kundenummer bør også vises for anonyme brukere. Det skal ikke være problemer i forbindelse med personvern.
Data anses som anonyme hvis det ikke lenger er
mulig, med de hjelpemidlene som med rimelighet kan tenkes å ha blitt brukt, å identifisere
enkeltpersoner i datasettet. Her er vi innafor.

@cbrevik cbrevik merged commit 1812792 into master Feb 7, 2022
@cbrevik cbrevik deleted the temp/customer-number-test branch February 7, 2022 08:10
@tormoseng
Copy link
Member

@cbrevik Selve vinsingen er fin i app og webshop. Oppdaterer seg ved innlogging/utlogging i app. På iOS/VoiceOver leses kundenummer opp fint, men på MacOS/Safari/Chrome + Android leses kundenummer opp som "tre millioner ...". Også nevnt i #469, så det er Android som bør merkes her.

@tormoseng
Copy link
Member

Skjermlesersaken er lagt i egen sak AtB-AS/kundevendt#257

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

Successfully merging this pull request may close these issues.

4 participants