-
Notifications
You must be signed in to change notification settings - Fork 1
Convert application from Angular to React/Next.js with MUI #468
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: dylan-smith <1508559+dylan-smith@users.noreply.github.com>
Co-authored-by: dylan-smith <1508559+dylan-smith@users.noreply.github.com>
Unit Test Results76 tests +70 76 ✅ +70 3s ⏱️ +3s Results for commit eda2d6b. ± Comparison against base commit b18b430. This pull request removes 6 and adds 76 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
UI Test Results1 files ±0 9 suites ±0 1h 48m 25s ⏱️ + 1h 45m 37s For more details on these failures, see this check. Results for commit eda2d6b. ± Comparison against base commit b18b430. ♻️ This comment has been updated with latest results. |
|
@copilot The unit tests all got deleted during the conversion to React. Please add the unit tests back, and ensure that the build at |
Co-authored-by: dylan-smith <1508559+dylan-smith@users.noreply.github.com>
Added 76 unit tests using Jest with the same test coverage as the original Angular tests. The tests cover:
The
Coverage: 82.48% statements, 85.71% branches, 83.73% lines. Commit: 0607fc0 |
Co-authored-by: dylan-smith <1508559+dylan-smith@users.noreply.github.com>
|
This PR has been deployed to a preview site for testing. This site will automatically be deleted when the PR is merged. Last deployed: 2026-01-06T05:04:58Z (UTC) |
Summary
This PR converts the DiveIntelligence dive planning application from Angular to React/Next.js with MUI (Material-UI) components.
Framework Changes
Unit Tests
CI Configuration Updates
dist/dive-intelligence/browser) to Next.js output (out)Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.