[Feature] SCA-exemption for payment cards and support reservations in Firebase Store

cbrevik commented 2 years ago

Origin

Tilstøtende sak i app AtB-AS/kundevendt#1034

Motivation

Enklere betalingsflyt for brukere, bankid-krav for lagret betalingskort er ikke mye bedre enn Vipps.

Hypotheses and assumptions

Flere brukere får bankid krav hver gang de betaler med betalingskort
En flyt uten bankid for lave beløp vil oppleves som mye raskere enn andre alternativer

Proposed solution

Test kort: https://shop.nets.eu/nb/web/partners/test-cards

Sende inn sca_exemption: true i reserve-kall
Hent reservations fra Firebase Store fremfor inmemory

Acceptance Criteria

[ ] Du får BankId-krav på test-kort som gir "challenge"
[ ] Du får ikke BankId-krav på test-kort som ikke gir "challenge"
[ ] Du får opp reservasjoner som før, men nå også dersom det gjøres en reservasjon i app eller annen tab.
[ ] Når du mottar billett så skal reservasjon bli borte.
[ ] Utdaterte reservasjoner skal ikke vises i oversikten (eldre enn 1 time, eller kansellerte reservasjoner)
[ ] Ved fullført kjøp skal du sendes tilbake til oversikten og se reservasjon som har gyldig betalingsmiddel og ordre-id. Når betaling er gjennomført og registrert skal den endres til vanlig aktiv billett.

Ser ut som kanskje disse gir challenge:

gorandalum commented 2 years ago

Ikke lenger nødvendig siden det blir fikset backend.

cbrevik commented 2 years ago

Tror en fortsatt må sende inn scaExcemption: true (for å enable SCA). Men det kan kanskje være en egen issue.

cbrevik commented 2 years ago

Må fortsatt sende inn den verdien fordi hvis vi enabler for alle så vil det brekke kjøp på gamle versjoner på app. Kan være at vi kunne skille på app og web

mikaelbr commented 2 years ago

Utvidet denne til å ha akseptansekriterier rundt teknisk endring på lagring av reservasjoner som nå er skrevet om til å fungere tilsvarende som på app for nettbutikk.

tormoseng commented 2 years ago

Denne er testet fint på staging + delvis på next.

@mikaelbr Testet dette på Next (https://atb-webshop-production--next-6spvs4ey.web.app/) og sjekket at /reserve ble sendt med _scaexcemption: true. Testet 7-dagerskjøp (294 kr) med både MasterCard og Visa. Begge kom med verifisering (MC-verifisering og BankID). Er det noe annet som må på plass for at dette skal fungere på webshop (SCA virker til å fungere på App med dagens prod-backend).

mikaelbr commented 2 years ago

@gorandalum du som har jobbet med sca på app her. Det skal vel være løst utelukkende på backend her så så lenge sca_excemption: true blir sendt så skal det være OK (for du bekreftet at det ble sendt, @tormoseng ?)

mikaelbr commented 2 years ago

Det er jo bare jeg som var forvirret og fortsatte forvirringen med å lese feil hele tiden. Fix: https://github.com/AtB-AS/webshop/pull/496

tormoseng commented 2 years ago

Re-testet OK på next med:

7-dagerskjøp (294 kr) med MasterCard
7-dagerskjøp (294 kr) med Visa
Klippekort barn (210 kr) med Visa
Klippekort voksen (410 kr) med MasterCard

AtB-AS / webshop