Skip to content
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

New Registration Form and many tech debt updates #1847

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

kylemh
Copy link
Member

@kylemh kylemh commented Jan 26, 2025

Description of changes

As requested by leadership, changing onboarding flow to use AirTable. Also update many deps, migrate lots of JS to TS, and resolve related issues.

Copy link

vercel bot commented Jan 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
operation-code ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2025 4:26am
storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2025 4:26am

@kylemh
Copy link
Member Author

kylemh commented Jan 26, 2025

Copy link

cypress bot commented Jan 26, 2025

operation_code    Run #4816

Run Properties:  status check passed Passed #4816  •  git commit 9b87646643: fix more form issues, finish e2e test, and update snapshots
Project operation_code
Branch Review update-registration-form
Run status status check passed Passed #4816
Run duration 03m 29s
Commit git commit 9b87646643: fix more form issues, finish e2e test, and update snapshots
Committer Kyle Holmberg
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 2
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 25
View all changes introduced in this branch ↗︎

Copy link

codeclimate bot commented Jan 29, 2025

Code Climate has analyzed commit 9b87646 and detected 3 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 3

The test coverage on the diff in this pull request is 76.0% (90% is the threshold).

This pull request will bring the total coverage in the repository to 80.1% (-1.0% change).

View more on Code Climate.

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