Conversation
BREAKING CHANGE: switch to iframe-wrapped experience builder app
There was a problem hiding this comment.
Pull request overview
This is a major breaking change that migrates the application from a Dojo-based architecture with Python backend scripts to a modern React + Vite application that wraps an ArcGIS Experience Builder app in an iframe. The PR removes extensive legacy code including Python scripts for data processing, Dojo widgets, tests, and build configurations, replacing it with a minimal React application that embeds the Experience Builder template.
Key Changes:
- Complete migration from Dojo to React with Vite as the build tool
- Removal of Python backend scripts and ArcGIS toolbox integration
- Introduction of Utah Design System components for header/footer
- Simplified architecture using iframe to embed ArcGIS Experience Builder
Reviewed changes
Copilot reviewed 113 out of 177 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| vite.config.ts | New Vite configuration with React plugin |
| tsconfig.json, tsconfig.browser.json, tsconfig.vite-config.json | TypeScript project configuration using composite projects |
| tailwind.config.js | Tailwind CSS configuration with UGRC preset |
| postcss.config.js | PostCSS configuration for Tailwind and Autoprefixer |
| pnpm-workspace.yaml | PNPM workspace configuration with build dependencies |
| package.json | Updated dependencies from Grunt/Dojo to React/Vite stack |
| src/main.tsx | Main React entry point with header, footer, and iframe setup |
| src/index.css | Base styles with Tailwind directives and custom fonts |
| src/vite-env.d.ts | TypeScript declarations for Vite environment |
| src/goeo_logo_light.svg | GOEO logo for light mode |
| scripts/* | Removed Python backend scripts for report generation and data processing |
| lib/* | Removed legacy HTML templates and configuration files |
| profiles/* | Removed Dojo build profiles |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
steveoh
reviewed
Dec 23, 2025
steveoh
reviewed
Dec 23, 2025
steveoh
previously approved these changes
Dec 23, 2025
Co-authored-by: steveoh <sgourley@utah.gov>
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
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.
BREAKING CHANGE: switch to iframe-wrapped experience builder app