Draft: switch AWS deployment from Serverless to CDK - #121
colorchestra wants to merge 36 commits into
Conversation
✅ Deploy Preview for serene-narwhal-9b7b1c ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for peaceful-raindrop-cb2462 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for serene-narwhal-9b7b1c ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for peaceful-raindrop-cb2462 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
…localhost (cherry picked from commit 69ff6ce)
jh0ker
left a comment
There was a problem hiding this comment.
Claudia und ich haben uns das mal vorgenommen. Alles in allem glaube ich gut. Kannst ja mal über meine Kommentare schauen und überlegen, was davon du dir noch vornehmen möchtest und was wir aufschieben.
| // 403 und 404 auf 200 und "Upps!"-Seite umbiegen; Originalverhalten | ||
| // langfristig vllt. so ändern, dass das nur App-Pfade betrifft und nicht z.B. API-Pfade |
There was a problem hiding this comment.
Ja ich glaub das könnte tatsächlich ein Problem sein. Wir haben einen Service Worker der die Assets cached, wenn wir da einmal ein Asset laden was fehlt, wird er denke ich die "Upps!"-Seite als Response cachen. Das ist dann schwierig, wieder loszuwerden. Ist erstmal kein großes Problem, weil der Offline-Modus hinter einem Feature Flag sitzt, und nur dann auch der Service Worker anspringen sollte.
Aber trotzdem fragwürdig, ob das sinnvoll ist, das jetzt 1:1 zu übernehmen. Wir haben ja jetzt auch z. B. die OAC neu hinzugefügt, also ein 100% "sauberes" Refactoring, wo wir die alten Bugs mitschleppen, ist ja nicht zwingend unser Ziel oder?
a12e982 to
2a84c66
Compare
…nt distribution and service worker
…invalidate cache after updating everything
No description provided.