Skip to content

fix(review): make live publishing a secondary confirmed action - #146

Closed
Ankit6149 wants to merge 15 commits into
masterfrom
fix/review-first-publishing
Closed

fix(review): make live publishing a secondary confirmed action#146
Ankit6149 wants to merge 15 commits into
masterfrom
fix/review-first-publishing

Conversation

@Ankit6149

@Ankit6149 Ankit6149 commented Aug 6, 2026

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

  • pure direct-publish guard covering stale, failed, needs-review, unapproved, over-limit, permission, connection, expiry, and manual-only states;
  • account identity and exact draft-revision confirmation contract;
  • API truth helper that accepts success only for a matching platform with a stable post ID or URL;
  • responsive secondary publishing panel and keyboard/Escape-accessible confirmation dialog;
  • primary copy/open and save actions remain available independently of connector readiness;
  • deterministic guard, confirmation, response-fixture, and UI-source tests.

Verification

Requires the complete frontend regression suite, production dependency audit, Next.js build, MCP tests, Python tests, and Vercel preview to pass 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:22am

Copy link
Copy Markdown
Owner Author

Superseded by #147, which carries the cleaned review-first publishing implementation and current verification work. Closing this duplicate branch so only one publishing PR remains active.

@Ankit6149 Ankit6149 closed this Aug 6, 2026
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