Kai Vandivier
Kai Vandivier
Thanks @asmirnov-style! I adapted mine a bit too: ```js const goOffline = () => { cy.log('**go offline**') // stub every request with a StaticResponse to simulate network error .then(() =>...
Looks great! This is a textual approval to avoid this getting merged accidentally 😁 ✅ I tested this pretty extensively both in a dev environment and a production build deployed...
You can test the boundary by logging in as a guest user (user guest, pass Guest123), which shouldn't have access to all apps -- also see the 'production env' test...
@amcgee I noticed that -- I refactored the app wrapper in my latest commit to work well with the changes that you have here: https://github.com/dhis2/app-platform/pull/754/files#diff-58fcbeed2ec3103f3d186cb172bf2da570e316e008fb614240d6aa0f72df7574 Taking a closer look now...
Also added handling for server versions < 35, as in https://github.com/dhis2/app-platform/blob/fix/pwa-handle-navigations/cli/src/lib/constructAppUrl.js#L8
Drafted while this is on the back-burner
I think concerns have been addressed and this is ready for rereview @amcgee
For me, this was happening when React Devtools was active in _both_ the browser extension and the standalone client -- closing one fixed the other 🥳 If you're going to...
Hm, I noticed that these new URLs end with a `/`, which seems to deviate from our current practice in some places -- here are some observations: * One place...