Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zip317 add propose command #917

Merged
merged 14 commits into from
Apr 17, 2024

Conversation

Oscar-Pepper
Copy link
Contributor

  • Adds propose and quicksend commands
  • Adds unimplemented do_propose and do_send_proposal lightclient methods

@Oscar-Pepper Oscar-Pepper marked this pull request as ready for review April 10, 2024 15:06
zancas
zancas previously approved these changes Apr 10, 2024
@Oscar-Pepper Oscar-Pepper marked this pull request as draft April 10, 2024 16:27
@fluidvanadium
Copy link
Contributor

this looks good. the only conflict is the changelog.

@Oscar-Pepper Oscar-Pepper marked this pull request as ready for review April 16, 2024 16:32
zingolib/src/commands.rs Show resolved Hide resolved
zingolib/src/commands/utils.rs Outdated Show resolved Hide resolved
zingolib/src/lightclient/send.rs Show resolved Hide resolved
fluidvanadium
fluidvanadium previously approved these changes Apr 17, 2024
Copy link
Contributor

@fluidvanadium fluidvanadium left a comment

Choose a reason for hiding this comment

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

Looks like comments have been addressed and PR is ready

zancas
zancas previously approved these changes Apr 17, 2024
@Oscar-Pepper Oscar-Pepper dismissed stale reviews from zancas and fluidvanadium via cea9691 April 17, 2024 15:24
@zancas zancas merged commit 25c4249 into zingolabs:dev Apr 17, 2024
17 checks passed
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.

3 participants