Skip to content

DBC22-5462: Upgrade to VITE and update npm packages#1367

Open
wmuldergov wants to merge 2 commits into
mainfrom
task/DBC22-5462-3
Open

DBC22-5462: Upgrade to VITE and update npm packages#1367
wmuldergov wants to merge 2 commits into
mainfrom
task/DBC22-5462-3

Conversation

@wmuldergov

Copy link
Copy Markdown
Collaborator

📝 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: You will need to update your .env file to update anything with REACT_APP_ with VITE_. For the openshift release, it will consume from either REACT_APP or VITE to reduce risk of issues. Eventually we can remove any references to REACT_APP once all variables updated in the vault.

🧪 How to Test (if required)

  1. Run the site locally using npm run start
  2. Try build the site locally using npm run build
  3. Try build the site using docker
  4. Try deploy the site in OpenShift
  5. For each of these, confirm that the site loads with no console errors
  6. Confirm that core site functionality still works the same as it did before (ie the sidepanels, various pages, etc)

🔍 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

@wmuldergov wmuldergov force-pushed the task/DBC22-5462-3 branch from 95c219a to 0439f2b Compare June 5, 2026 18:37
// import { HeaderHeightContext } from '../App.js';
import EmergencyAlert from "../Components/shared/EmergencyAlert.js";
// import { HeaderHeightContext } from '../App';
import EmergencyAlert from "../Components/shared/EmergencyAlert";
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.

1 participant