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

Update state section on test-snaps #3014

Merged
merged 2 commits into from
Jan 14, 2025
Merged

Update state section on test-snaps #3014

merged 2 commits into from
Jan 14, 2025

Conversation

Mrtenz
Copy link
Member

@Mrtenz Mrtenz commented Jan 14, 2025

Some HTML IDs were duplicated between the state and manage state sections on test-snaps, and there was a missing button as well. I've fixed both issues in this PR.

@Mrtenz Mrtenz requested a review from a team as a code owner January 14, 2025 15:46
@Mrtenz Mrtenz changed the title Remove duplicate IDs from test-snaps Remove duplicate IDs from test-snaps Jan 14, 2025
@Mrtenz Mrtenz changed the title Remove duplicate IDs from test-snaps Update state section on test-snaps Jan 14, 2025
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.76%. Comparing base (1ab6bc9) to head (1056403).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3014   +/-   ##
=======================================
  Coverage   94.76%   94.76%           
=======================================
  Files         499      499           
  Lines       10906    10906           
  Branches     1674     1674           
=======================================
  Hits        10335    10335           
  Misses        571      571           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Mrtenz Mrtenz force-pushed the mrtenz/duplicate-ids branch from 3693b1c to 1056403 Compare January 14, 2025 18:27
@Mrtenz Mrtenz merged commit ba4fd7f into main Jan 14, 2025
166 checks passed
@Mrtenz Mrtenz deleted the mrtenz/duplicate-ids branch January 14, 2025 20:43
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