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

Signer bug fix #129

Merged
merged 9 commits into from
May 24, 2023
Merged

Signer bug fix #129

merged 9 commits into from
May 24, 2023

Conversation

SupremoUGH
Copy link
Contributor

@SupremoUGH SupremoUGH commented May 18, 2023

Fixes the wallet AssetSpent bug. Goes together with Manta-Network/manta-rs#347 and Manta-Network/manta-rs#349


Before we can merge this PR, please make sure that all the following items have been checked off:

  • Linked to an issue with discussion and accepted design OR have an explanation in the PR that describes this work.
  • Added one line describing your change in CHANGELOG.md and added the appropriate L- label to the PR.
  • Re-reviewed Files changed in the GitHub PR explorer.

@SupremoUGH SupremoUGH added L-fixed fixed Changelog: add these changes to the `fixed` section of the changelog P-high Priority: High C-bug Category: Issues documenting a bug labels May 18, 2023
@SupremoUGH SupremoUGH marked this pull request as ready for review May 24, 2023 14:40
@DanielZhangReal DanielZhangReal requested review from DanielZhangReal and ferrell-code and removed request for DanielZhangReal May 24, 2023 14:44
@SupremoUGH SupremoUGH merged commit 5e588d8 into signer_extension May 24, 2023
@SupremoUGH SupremoUGH deleted the signer_bug_fix branch May 24, 2023 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: Issues documenting a bug L-fixed fixed Changelog: add these changes to the `fixed` section of the changelog P-high Priority: High
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants