Skip to content

fix(review): complete review-first publishing flow - #147

Open
Ankit6149 wants to merge 21 commits into
masterfrom
fix/review-first-publishing-final
Open

fix(review): complete review-first publishing flow#147
Ankit6149 wants to merge 21 commits into
masterfrom
fix/review-first-publishing-final

Conversation

@Ankit6149

Copy link
Copy Markdown
Owner

Product outcome

Keep Review centered on editing, save, copy/open, and export. Direct connector publishing becomes a separate secondary action with explicit account and revision confirmation.

Included

  • direct-publish guard for stale, failed, needs-review, unapproved, over-limit, permission, connection, expiry, manual-only, and unverified connector states;
  • platform, connected account, exact revision, and draft-state confirmation before a live action;
  • success only after a matching destination response includes a stable post ID or URL;
  • responsive secondary publishing panel and keyboard/Escape-accessible confirmation dialog;
  • manual copy/open and export remain complete when no live connector is available;
  • regression coverage for the full guard matrix, API response truth, confirmation contract, action priority, and accessible UI structure.

A temporary CI job applies the final two compatibility-safe source adjustments on this clean PR event. It will be removed and the normal CI workflow restored before merge.

Closes #48.

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
signal-flow-studio Ready Ready Preview Aug 6, 2026 5:30am

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.

[P2][Plan 5] SF-024 — Make publishing secondary to review, save, and export

1 participant