Skip to content

Sideloading tutorial#147

Merged
sgwilym merged 5 commits intomacromaniafrom
sideloading-tutorial
Aug 5, 2025
Merged

Sideloading tutorial#147
sgwilym merged 5 commits intomacromaniafrom
sideloading-tutorial

Conversation

@sgwilym
Copy link
Collaborator

@sgwilym sgwilym commented Aug 4, 2025

Adds a tutorial for sideloading. Create a store, create a drop of an area of that store, ingest the drop, everybody's happy.

@AljoschaMeyer
Copy link
Collaborator

Store tutorial has broken refs in its summary (the second one should definitely not be broken anymore with this PR):

image

@AljoschaMeyer
Copy link
Collaborator

I think the sideloading tutorial would benefit a lot from a quick refresher of what is going on. A link to the sideloading spec, an reminder of what a "drop" is again (in particular because Drop and drop have preestablished meaning in Rust).

@AljoschaMeyer
Copy link
Collaborator

Likewise, Instantiated the export store can be confusing I think: what is an "export store"? Why do we need two stores? What is going on?

@AljoschaMeyer
Copy link
Collaborator

image

I genuinely got confused by this for a second. What are the "contents" of a drop, to be "stored" in a vec? My mind went to a vec of entries first.
(The code block has a similarly confusing comment: // Create a drop and place its contents in a Vec<u8>)

@AljoschaMeyer
Copy link
Collaborator

TODO: NICE ENTRY PRINTING!: :#? formatting is probably good enough maybe? I think it might be fine to cheat and manually adjust the purported console output, putting all crypto bytes into a single line but leaving the nice formatting of struct-field-per-line.

@AljoschaMeyer
Copy link
Collaborator

image

typo
(also, is "instantiated" really better than "created"?)

@sgwilym
Copy link
Collaborator Author

sgwilym commented Aug 5, 2025

Thank you! Tried to address all of these except the pretty printing, lmk what you think.

@AljoschaMeyer
Copy link
Collaborator

lgtm now

@sgwilym sgwilym merged commit 23b2dbb into macromania Aug 5, 2025
3 checks passed
@sgwilym sgwilym deleted the sideloading-tutorial branch August 5, 2025 14:46
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.

2 participants