Skip to content

refactor: remove Encrypted note type and update related documentation#2315

Merged
bobbinth merged 4 commits into0xMiden:nextfrom
swaploard:Removing-NoteType--Encrypted-(#2310)
Jan 20, 2026
Merged

refactor: remove Encrypted note type and update related documentation#2315
bobbinth merged 4 commits into0xMiden:nextfrom
swaploard:Removing-NoteType--Encrypted-(#2310)

Conversation

@swaploard
Copy link
Contributor

Remove Encrypted NoteType variant (#2310)

Changes

  • Removed the NoteType::Encrypted enum variant.
  • Updated error messages and formatting to no longer reference Encrypted.
  • Adjusted tests and type matchers affected by the removal.
  • Ensured linting and builds pass without the encrypted variant.

Copy link
Contributor

@PhilippGackstatter PhilippGackstatter 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, thanks!

@PhilippGackstatter PhilippGackstatter linked an issue Jan 20, 2026 that may be closed by this pull request
2 tasks
Copy link
Contributor

@bobbinth bobbinth 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! Thank you!

@bobbinth bobbinth merged commit 6bde539 into 0xMiden:next Jan 20, 2026
15 checks passed
@swaploard swaploard deleted the Removing-NoteType--Encrypted-(#2310) branch January 22, 2026 04:26
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.

Remove NoteType::Encrypted

3 participants