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

DEV-5640: Migrate DReason to TypeScript #1694

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nrh-cklimas
Copy link
Collaborator

Please fill out each section even if it's just with "N/A"

Plan? (if this draft/incomplete indicate what you intend to do and how)

Creating review app for design review.

What's this PR do?

Why are we doing this? How does it help us?

Are there detailed, specific, step-by-step testing instructions in the associated ticket?

Did this PR make changes to the user interface that may require changes to the user-facing docs?

Have automated unit tests been added? If not, why?

How should this change be communicated to end users?

Are there any smells or added technical debt to note?

Has this been documented? If so, where?

What are the relevant tickets? Add a link to any relevant ones.

Do any changes need to be made before deployment to production (adding environment variables, for example)? If so, open a ticket and link it to the ticket for this PR and list it here:

Are there next steps? If so, what? Have tickets been opened for them? List next-step tickets here:

@x110dc x110dc temporarily deployed to rev-engine-dev-5640-con-texybg January 9, 2025 21:16 Inactive
Copy link

cypress bot commented Jan 9, 2025

NRE    Run #7756

Run Properties:  status check failed Failed #7756  •  git commit 2dbab8ec50: DEV-5640: Migrate DReason to TypeScript
Project NRE
Branch Review DEV-5640-convert-reason-for-giving-to-typescript
Run status status check failed Failed #7756
Run duration 04m 00s
Commit git commit 2dbab8ec50: DEV-5640: Migrate DReason to TypeScript
Committer Chris Klimas
View all properties for this run ↗︎

Test results
Tests that failed  Failures 2
Tests that were flaky  Flaky 0
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 184
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/01-donationPage.cy.js • 2 failed tests • Cypress integration tests

View Output

Test Artifacts
User flow: canceling contribution > happy path Screenshots
User flow: canceling contribution > double-clicking the cancel button Screenshots

Base automatically changed from DEV-5617-fix-reason-for-giving-validation-error to main January 10, 2025 15:24
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.

2 participants