Skip to content

Correct manageBuyOffer cancel parameter in trustline removal - #91

Merged
kaankacar merged 1 commit into
mainfrom
fix-managebuyoffer-cancel-param
Aug 13, 2026
Merged

Correct manageBuyOffer cancel parameter in trustline removal#91
kaankacar merged 1 commit into
mainfrom
fix-managebuyoffer-cancel-param

Conversation

@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

Follow-up to #90, addressing both points from Copilot's review there: manageBuyOffer cancels with buyAmount: "0" (not amount), and the preflight snippet now notes that liquidity-pool usage (precondition 4) is not visible on the balance line and surfaces as op_cannot_delete at submit.

Copilot AI balanced review requested due to automatic review settings August 13, 2026 14:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Corrects trustline-removal guidance for Stellar offer cancellation and liquidity-pool constraints.

Changes:

  • Uses buyAmount: "0" for manageBuyOffer cancellation.
  • Clarifies that liquidity-pool usage surfaces as op_cannot_delete.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kaankacar
kaankacar merged commit e0133e8 into main Aug 13, 2026
7 checks passed
@kaankacar
kaankacar deleted the fix-managebuyoffer-cancel-param branch August 13, 2026 14:11
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-13 14:11 UTC

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