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.

Sign up for GitHub

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

Jump to bottom

feat: 1245 extraction and storage cypress test #1273

Merged
merged 35 commits into from
Jul 12, 2024
Merged

feat: 1245 extraction and storage cypress test #1273

merged 35 commits into from
Jul 12, 2024

Conversation

ngunner15
Copy link
Collaborator

@ngunner15 ngunner15 commented Jun 26, 2024 •
edited by github-actions bot
Loading

Description

Closes #1245
Testing for this step
image

Changelog

New

  • extraction and storage cypress test

Changed

  • cypress.config
  • A class reg form fixture

Removed

  • None

How was this tested?

  • 🧠 Not needed
  • 👀 Eyeball
  • 🤖 Added tests

What gif/image best describes this PR or how it makes you feel?

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@ngunner15 ngunner15 added front-end front-end related task test test related task labels Jun 26, 2024
@ngunner15 ngunner15 self-assigned this Jun 26, 2024
@ngunner15 ngunner15 linked an issue Jun 26, 2024 that may be closed by this pull request
Extraction and Storage Cypress Test #1245
Closed
@ngunner15 ngunner15 marked this pull request as ready for review June 26, 2024 23:02
craigyu
craigyu approved these changes Jul 9, 2024
View reviewed changes
Copy link
Collaborator

@craigyu craigyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's get it in then investigate the flakiness in a separate task

@ngunner15
Copy link
Collaborator Author

Reopening this PR as there was a fix by Ricardo(update username password)

@ngunner15 ngunner15 reopened this Jul 12, 2024
RMCampos
RMCampos approved these changes Jul 12, 2024
View reviewed changes
Copy link
Contributor

@RMCampos RMCampos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ngunner15 ngunner15 merged commit fe56738 into main Jul 12, 2024
28 checks passed
@ngunner15 ngunner15 deleted the feat/1245-extraction-and-storage-cypress-test branch July 12, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RMCampos RMCampos RMCampos approved these changes

@craigyu craigyu craigyu approved these changes

@mgaseta mgaseta Awaiting requested review from mgaseta mgaseta is a code owner

@DerekRoberts DerekRoberts Awaiting requested review from DerekRoberts DerekRoberts is a code owner

@SLDonnelly SLDonnelly Awaiting requested review from SLDonnelly SLDonnelly is a code owner

Assignees

@ngunner15 ngunner15

Labels
front-end front-end related task test test related task
Projects
SPAR Sprints | Goal: Improving the SPAR user experience; CONSEP seed testing activities usability testing; finish lo-fi wireframes for additional seed testing activities (family lot, reporting)
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Extraction and Storage Cypress Test
4 participants
@ngunner15 @DerekRoberts @RMCampos @craigyu