Skip to content

Conversation

@fnando
Copy link
Member

@fnando fnando commented Sep 15, 2025

What

Add two commands:

  • stellar tx new liquidity-pool-deposit
  • stellar tx op add liquidity-pool-deposit

Why

#2129

Known limitations

N/A

@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Sep 15, 2025
Copy link
Collaborator

@elizabethengelman elizabethengelman left a comment

Choose a reason for hiding this comment

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

Looks good! I wonder if its worth adding a test too

@fnando
Copy link
Member Author

fnando commented Sep 15, 2025

@elizabethengelman I thought about it, but setting up a liquidity pool would be tricky.

@elizabethengelman
Copy link
Collaborator

setting up a liquidity pool would be tricky.

@fnando that makes sense. I think that we created a liquidity pool for the ledger entry fetch integration tests, but I'm not sure if it's worth it

@fnando fnando merged commit c37bc36 into main Sep 15, 2025
43 of 49 checks passed
@fnando fnando deleted the liquidity-pool-deposit branch September 15, 2025 20:30
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants