Skip to content

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
@codecov
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.

3 participants