-
Notifications
You must be signed in to change notification settings - Fork 0
Promote v3 to Production #267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR promotes the v3 branch to production, representing a complete rewrite of the raster data discovery application from Dojo/Grunt to React/Vite/TypeScript. The modernization includes a comprehensive Playwright test suite, improved state management with XState, and enhanced developer tooling.
Key Changes:
- Complete migration from Dojo to React 19 with TypeScript
- Modern build system using Vite instead of Grunt
- New state management using XState
- Comprehensive Playwright E2E test suite with shared test helpers
- Updated styling with Tailwind CSS
Reviewed Changes
Copilot reviewed 162 out of 196 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated to React 19, Vite 7, and modern tooling; removed legacy Dojo dependencies |
| vite.config.ts | New Vite configuration with Vitest setup |
| tsconfig.*.json | TypeScript project references configuration for browser and Vite contexts |
| src/services/*.ts | New service layer for API calls, state management, and business logic |
| src/components/*.tsx | React components replacing legacy Dojo widgets |
| tests/**/*.spec.ts | Comprehensive Playwright test suite organized by feature category |
| tests/fixtures/test-helpers.ts | Shared test utilities reducing duplication across test files |
| src/hooks/*.ts | Custom React hooks for context access |
| src/contexts/*.tsx | React context providers for global state |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
are these conflicts something we can resolve on the dev branch? |
Start fresh BREAKING CHANGE: This is a total rewrite of the application
Closes #217
So that we can get checks running on the dev branch.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
I fixed the conflicts so that this PR is merge-able now. The only thing that is holding this back is #271. |
|
@copilot Add a light background to the qualtrics survey that doesn't change so that it has a suitable contrast in both light and dark modes. |
…oading via map popup
|
I've implemented temporary workarounds for these issues: I propose that we merge this and work on deploying to production next week. @rkelson Do you want to take one more look at https://raster.dev.utah.gov/ and verify that it is ready for production? |
I'll likely complete this merge manually using the CLI since there's been some divergence between the two branches. This is just a place for @rkelson and anyone else to give final feedback before a production release.
You can test the staging version here: https://raster.dev.utah.gov/
Recent release notes for staging releases can be found here: https://github.com/agrc/raster/releases