Skip to content

fix: add checkmark prefix to created/synced output lines and fix progress bar counter#109

Merged
greghuels merged 1 commit into
mainfrom
greg.huels/fix-checkmarks
Jul 20, 2026
Merged

fix: add checkmark prefix to created/synced output lines and fix progress bar counter#109
greghuels merged 1 commit into
mainfrom
greg.huels/fix-checkmarks

Conversation

@greghuels

@greghuels greghuels commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Checkmarks were missing for both flags created and synced to indicate that the submission was successful. I had this at one point, but I think it regressed after the React refactor.

Summary

  • Prefix successful Created and Synced output lines with a green checkmark so they are visually distinct from the overwrite warnings that precede them
  • Fix the progress bar checkmark counter to track both created and synced flags (was only counting created, showing 0 when all flags were syncs)
  • Rename ProgressBarStats.created to saved to reflect that it represents all successful submissions
Screenshot 2026-07-20 at 2 30 22 PM

Test plan

  • Run a migration with a mix of new and existing flags and verify checkmark appears on each successful line
  • Confirm the progress bar checkmark counter increments for both created and synced flags

…ress bar counter

- Prefix successful Created/Synced output lines with a green ✓
- Progress bar ✓ counter now tracks both created and synced flags (renamed field to `saved`)

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@greghuels
greghuels marked this pull request as ready for review July 20, 2026 19:31
@greghuels
greghuels requested a review from a team as a code owner July 20, 2026 19:31
@greghuels
greghuels requested review from mark-caii and vjfridge July 20, 2026 21:02
@greghuels
greghuels merged commit 7eef7ab into main Jul 20, 2026
4 checks passed
@greghuels
greghuels deleted the greg.huels/fix-checkmarks branch July 20, 2026 21:04
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