Skip to content

return the review IDs in the integration outcome#7823

Merged
estib-vega merged 2 commits intomasterfrom
e-branch-64
Mar 25, 2025
Merged

return the review IDs in the integration outcome#7823
estib-vega merged 2 commits intomasterfrom
e-branch-64

Conversation

@estib-vega
Copy link
Copy Markdown
Contributor

@estib-vega estib-vega commented Mar 25, 2025

Integration outcome returns information about the review IDs that should be closed, if any

If there are any review IDs attached to the archived branches, return them to the frontend in order to close them.
Close the reviews of archived branches by using the returned review ids
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 25, 2025 6:40pm
gitbutler-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 25, 2025 6:40pm

@github-actions github-actions bot added rust Pull requests that update Rust code @gitbutler/desktop labels Mar 25, 2025
@estib-vega estib-vega changed the title e-branch-64 return the review IDs in the integration outcome Mar 25, 2025
@estib-vega estib-vega enabled auto-merge March 25, 2025 18:40
@estib-vega estib-vega disabled auto-merge March 25, 2025 18:42
Copy link
Copy Markdown
Member

@krlvi krlvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@estib-vega estib-vega merged commit fadaf7f into master Mar 25, 2025
19 checks passed
@estib-vega estib-vega deleted the e-branch-64 branch March 25, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@gitbutler/desktop rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants