Skip to content

chore: Rename NoteMetadata to PartialNoteMetadata#2887

Open
PhilippGackstatter wants to merge 3 commits intopgackst-multiple-note-attachments-followupfrom
pgackst-partial-note-metadata
Open

chore: Rename NoteMetadata to PartialNoteMetadata#2887
PhilippGackstatter wants to merge 3 commits intopgackst-multiple-note-attachments-followupfrom
pgackst-partial-note-metadata

Conversation

@PhilippGackstatter
Copy link
Copy Markdown
Contributor

@PhilippGackstatter PhilippGackstatter commented May 8, 2026

Renames NoteMetadata to PartialNoteMetadata and NoteMetadataHeader to NoteMetadata as motivated in #2874 (comment).

Small change sneaked in: Simplifies BatchNoteTree and BlockNoteTree constructors to take NoteHeader instead of NoteId + NoteMetadata.

closes #2874

@PhilippGackstatter PhilippGackstatter added the pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority label May 8, 2026
@PhilippGackstatter PhilippGackstatter force-pushed the pgackst-partial-note-metadata branch from f429506 to a8daf68 Compare May 8, 2026 08:48
@PhilippGackstatter PhilippGackstatter marked this pull request as ready for review May 8, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-from-maintainers PRs that come from internal contributors or integration partners. They should be given priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant