Skip to content

DBC22-5342: check version and refresh every hour#1320

Open
bcgov-brwang wants to merge 1 commit into
mainfrom
bugfix/DBC22-5342
Open

DBC22-5342: check version and refresh every hour#1320
bcgov-brwang wants to merge 1 commit into
mainfrom
bugfix/DBC22-5342

Conversation

@bcgov-brwang
Copy link
Copy Markdown
Collaborator

@bcgov-brwang bcgov-brwang commented May 6, 2026

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. Deploy to dev
  2. Open a frontend page, and leave it open
  3. Make a new deploy to dev
  4. Verify the existing frontend still works without refresh and backend does not have nginx error logs complaining similar to the following:
    2025/12/24 21:50:14 [error] 83#83: *4088040 open() "/usr/share/nginx/html/static/js/510.1adae997.chunk.js" failed (2: No such file or directory),

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

@bcgov-brwang bcgov-brwang force-pushed the bugfix/DBC22-5342 branch 2 times, most recently from cd95f93 to 06763cd Compare May 7, 2026 19:01
@bcgov-brwang bcgov-brwang changed the title DBC22-5342: check version and refresh DBC22-5342: check version and refresh every week May 7, 2026
@bcgov-brwang bcgov-brwang requested a review from wmuldergov May 7, 2026 19:08
@bcgov-brwang bcgov-brwang marked this pull request as ready for review May 7, 2026 19:08
Comment thread src/frontend/src/App.js Outdated
DBC22-5342: refresh page when user leave the page for a long time in case new deployment

DBC22-5342: refresh the page every hour
@wmuldergov wmuldergov changed the title DBC22-5342: check version and refresh every week DBC22-5342: check version and refresh every hour May 12, 2026
@wmuldergov wmuldergov added the ready for review The PR is ready for review label May 13, 2026
Copy link
Copy Markdown
Collaborator

@wmuldergov wmuldergov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't work when I deploy to OpenShift as the version.json doesn't exist. I believe StaticBuild will need to be updated so it goes into the correct location or generate-version.js needs a different path.
The build shows it generated version.json so I presume it's just going to the wrong location

@wmuldergov wmuldergov added pending rework If the PR needs some re-work by the developer and removed ready for review The PR is ready for review labels May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending rework If the PR needs some re-work by the developer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants