Skip to content

Conversation

@grayside
Copy link
Contributor

@grayside grayside commented May 3, 2023

Progress towards #64

This change is expected to restore frontend UI loading after firebase deploy completes. It does not address the gap between terraform apply and firebase deploy.

@grayside grayside self-assigned this May 3, 2023
@grayside grayside changed the title fix: frontend loads once deploy completes fix: prevent website not found errors after firebase deploy completes May 3, 2023
@glasnt glasnt force-pushed the purge-frontend-cache-on-deploy branch from 772c2dd to b5731d0 Compare May 4, 2023 01:22
glasnt added 2 commits May 4, 2023 14:24
…udPlatform/terraform-dynamic-python-webapp into purge-frontend-cache-on-deploy
@glasnt
Copy link
Collaborator

glasnt commented May 4, 2023

I updated the code to correctly validate that the URL should error (the assertResponseContains helper presumes a healthy URL, so that was the test failure). My go isn't the strongest so it could be improved. If you're okay with my change, mark this as "ready for review" and I'll approve and merge.

@grayside
Copy link
Contributor Author

grayside commented May 4, 2023

I refactored the code to have an assertErrorResponseContains function.

@grayside grayside marked this pull request as ready for review May 4, 2023 23:43
@grayside grayside requested review from a team, donmccasland and glasnt as code owners May 4, 2023 23:43
@glasnt glasnt merged commit 6d34e1e into main May 5, 2023
@glasnt glasnt deleted the purge-frontend-cache-on-deploy branch May 5, 2023 00:34
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.

2 participants