Promote DBot Dev #155
Merged
Promote DBot Dev #155
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the Vista application from Create React App to Vite, updates React from v18 to v19, and standardizes the build tooling and dependencies. The migration includes updating environment variable handling, test framework changes, and build configuration updates.
- Migration from Create React App to Vite build system
- Update from React v18 to v19 with related dependency updates
- Switch from Jest to Vitest for testing framework
Reviewed Changes
Copilot reviewed 28 out of 32 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.js | New Vite configuration file replacing Create React App build system |
| package.json | Updated dependencies and scripts for Vite/Vitest migration |
| src/setupTests.js | Removed Jest-specific test setup |
| src/components/esrijs/MapView.test.jsx | Updated test syntax from Jest to Vitest |
| src/App.jsx | Updated environment variable access and import path |
| eslint.config.mjs | Simplified ESLint configuration using @ugrc/eslint-config |
| .env files | Renamed environment variables from REACT_APP_ to VITE_APP_ prefix |
| index.html | Updated HTML template for Vite with manual script import |
| .github/workflows/ | Updated CI/CD workflows to use pnpm instead of npm |
Member
Author
|
@steveoh Pinging you on this in case you missed the notification in the dbot avalanche. |
steveoh
approved these changes
Aug 1, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A lot of upgrades and standardizations. Waiting to hear back from DTS on Vista Citrix versions so that I know if I can do other arcgis-related upgrades (see note at the bottom of README).