Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

DEVPROD-3326: Reset edit host modal state #2275

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

sophstad
Copy link
Contributor

DEVPROD-3326

Description

  • Use a key prop associated with the host ID to reset the modal's state when working with different hosts.

Testing

  • Unable to reproduce as described in ticket with these changes applied

@sophstad sophstad requested a review from a team February 26, 2024 19:08
Copy link

cypress bot commented Feb 26, 2024

4 flaky tests on run #15894 ↗︎

0 547 10 0 Flakiness 4
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

Add key to reset component state
Project: Spruce Commit: cd97094d6c
Status: Passed Duration: 20:12 💡
Started: Feb 26, 2024 7:12 PM Ended: Feb 26, 2024 7:33 PM

Review all test suite changes for PR #2275 ↗︎

Copy link
Contributor

@khelif96 khelif96 left a comment

Choose a reason for hiding this comment

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

Great job and neat solution!

@sophstad sophstad merged commit 51becdb into evergreen-ci:main Feb 26, 2024
5 of 6 checks passed
@sophstad sophstad deleted the DEVPROD-3326 branch February 26, 2024 22:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants