Skip to content

chore(docs): add structs in map doc#17870

Merged
AztecBot merged 1 commit intonextfrom
ek/chore/add-structs-in-maps-doc
Oct 27, 2025
Merged

chore(docs): add structs in map doc#17870
AztecBot merged 1 commit intonextfrom
ek/chore/add-structs-in-maps-doc

Conversation

@sklppy88
Copy link
Contributor

No description provided.

@sklppy88 sklppy88 marked this pull request as draft October 22, 2025 13:52
@critesjosh
Copy link
Contributor

I am wondering if it makes more sense to put this content in docs/docs/developers/docs/guides/smart_contracts/how_to_implement_custom_notes.md. Then we could dedup some of the content, wdyt?

@sklppy88
Copy link
Contributor Author

I am wondering if it makes more sense to put this content in docs/docs/developers/docs/guides/smart_contracts/how_to_implement_custom_notes.md. Then we could dedup some of the content, wdyt?

Ah good point, how about the public stuff though ?

@critesjosh
Copy link
Contributor

ah yea, i think the public stuff makes sense on the current page

@sklppy88 sklppy88 marked this pull request as ready for review October 27, 2025 11:26
@sklppy88 sklppy88 requested a review from critesjosh October 27, 2025 11:27
Co-authored-by: Esau <esau@aztecprotocol.com>
@AztecBot AztecBot force-pushed the ek/chore/add-structs-in-maps-doc branch from 157cea3 to 2abaa63 Compare October 27, 2025 11:48
@AztecBot AztecBot enabled auto-merge October 27, 2025 11:48
Copy link
Contributor

@critesjosh critesjosh 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. just flagging that the syntax will need to be updated when this PR merges

@AztecBot AztecBot added this pull request to the merge queue Oct 27, 2025
Merged via the queue into next with commit 0e4a1e7 Oct 27, 2025
15 checks passed
@AztecBot AztecBot deleted the ek/chore/add-structs-in-maps-doc branch October 27, 2025 16:46
ludamad pushed a commit that referenced this pull request Dec 16, 2025
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