breaking-change: flag for sca excemption to reserve

AtB-AS / webshop

AtB Webshop / AtB Nettbutikk

https://nettbutikk.atb.no

European Union Public License 1.2

0 stars 1 forks source link

breaking-change: flag for sca excemption to reserve #487

Closed mikaelbr closed 2 years ago

mikaelbr commented 2 years ago

breaking-change: Requires update-reservation Cloud Function, new ticketing service.

Updates Webshop to send in sca_excemption when not Vipps payment
Changes to use reservation stored in Firebase Store instead of locally in memory. Currently not taking full advantage of that except it is persistent between instances (app/webshop) and persistent when refreshing site. A good follow-up of this task is to better handle cancellation/timeouts etc and utilize reservation persistence more.
Adds payment type to the reservation as this has been an expressed wish previously but not available in the data before.

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit 7b1d3fd):

https://atb-webshop-staging--pr487-mikael-sca-7zgotium.web.app

_{(expires Wed, 09 Mar 2022 14:05:13 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

gorandalum commented 2 years ago

Det ser ikke helt bra ut nei. Behøver ikke bruke mer tid i å nøste opp i dette, vi klarer oss med litt duplisering i modellen :+1:

mikaelbr commented 2 years ago

Skal bare skrive om kjapt for å gjøre det litt bedre. Dette er i utgangspunktet ikke bra, og må prøve å se om det er mulig å gjøre noen optimaliseringer her. Slik TEA er nå vil jo state oppdatere alle forgreninger hele tiden uten at VDOM oppdaterer men all CPU-arbeid gjøres fremdeles. Og det gjøres X ganger i sekundet ut i fra for hver subscription på posix/date. Dette uten noe slags heuristikk rundt arbeid som faktisk ikke trengs å gjøres. Får ikke løst det problemet her nå, men må gjøre noe små sjekk om det er mulig å gjøre det levelig.

gorandalum commented 2 years ago

Jeg som har bestilt en M1 Max fordi batteriet var blitt så dårlig på min MBP de siste månedene 🤔

mikaelbr commented 2 years ago

Tror jeg skal bare kjapt utforske oppdatering av state på tick fremfor på derivert state. Men jeg gjør det som en egen PR. For denne PR-en gjør det litt mer, men det er ikke synderen. I vanlig reaktive system ville jeg sagt at å gjøre det inn mot state er en kilde til stale state og inconsistant data, men i dette tilfellet ser det ut som det er en nødvendig onde.