Skip to content

SF-3931 Show draft running or failed in the menu - #4127

Open
pmachapman wants to merge 3 commits into
masterfrom
fix/SF-3931
Open

pmachapman wants to merge 3 commits into
masterfrom
fix/SF-3931

Conversation

@pmachapman

@pmachapman pmachapman commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

This PR makes the draft generation menu item act like the sync menu item by showing that a draft is in progress, or showing that a draft has failed.

The draft generating animation looks like:

Recording 2026-09-21 145840

(it is SVG so can be tweaked however you like)


This change is Reviewable

@pmachapman pmachapman added the will require testing PR should not be merged until testers confirm testing is complete label Sep 21, 2026
@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.96%. Comparing base (3f221d3) to head (777d427).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4127      +/-   ##
==========================================
+ Coverage   80.94%   80.96%   +0.01%     
==========================================
  Files         664      664              
  Lines       42182    42224      +42     
  Branches     6951     6955       +4     
==========================================
+ Hits        34146    34188      +42     
  Misses       6870     6870              
  Partials     1166     1166              

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

@pmachapman
pmachapman deployed to screenshot_diff September 21, 2026 03:08 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

📸 Screenshot diff deployed! (2 changes)

View the visual diff at: https://pr-4127--sf-screenshot-diffs.netlify.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will require testing PR should not be merged until testers confirm testing is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant