Skip to content

Conversation

@batconjurer
Copy link
Collaborator

Describe your changes

When using the ledger client, we update witness maps manually. We now add notes to the witness map if there were successfully trial decrypted.

Checklist before merging

  • If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
    • This will require 2 reviewers to approve the changes
  • If this PR requires changes to the docs or specs, a corresponding PR is opened in the namada-docs repo
    • Relevant PR if applies:
  • If this PR affects services such as namada-indexer or namada-masp-indexer, a corresponding PR is opened in that repo
    • Relevant PR if applies:

@batconjurer batconjurer requested a review from sug0 June 10, 2025 13:01
@github-actions github-actions bot added the breaking:api public API breaking change label Jun 10, 2025
@sug0
Copy link
Collaborator

sug0 commented Jun 11, 2025

just missing a changelog

tzemanovic added a commit that referenced this pull request Jul 16, 2025
* origin/bat/dont-update-witness-maps-of-unowned-notes:
  changelog
  Don't update witness maps of unowned notes
@tzemanovic tzemanovic merged commit 20fdd86 into main Jul 17, 2025
22 of 24 checks passed
@tzemanovic tzemanovic deleted the bat/dont-update-witness-maps-of-unowned-notes branch July 17, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking:api public API breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants