-
Notifications
You must be signed in to change notification settings - Fork 256
Data Attachment Docs #405
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Data Attachment Docs #405
Conversation
✅ Deploy Preview for nimble-elf-d9d491 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
DennisOchulor
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if it's really necessary, but the attachment change event could use a little section about it.
Co-authored-by: DennisOchulor <113277083+DennisOchulor@users.noreply.github.com>
reference/latest/src/main/java/com/example/docs/attachment/Stamina.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/attachment/Stamina.java
Outdated
Show resolved
Hide resolved
Co-authored-by: DennisOchulor <113277083+DennisOchulor@users.noreply.github.com>
DennisOchulor
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now 👍
ekulxam
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I already left most of my comments in https://discord.com/channels/507304429255393322/1440182041952915518, just have a few questions.
its-miroma
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clear article, but a bit wordy. I've simplified and shortened some parts. Also, consider adding as many code blocks to the reference as possible, especially since this API is experimental and might change in the future.
Co-authored-by: Miroma <its.miroma@proton.me>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor fixes.
I still think that as much code as possible should be moved to the reference
Co-authored-by: Miroma <its.miroma@proton.me>
its-miroma
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I capitalized Data Attachment. LGTM
reference/latest/src/main/java/com/example/docs/attachment/ExampleModAttachments.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/attachment/ExampleModAttachments.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/attachment/ExampleModAttachments.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/attachment/ExampleModAttachments.java
Outdated
Show resolved
Hide resolved
reference/latest/src/main/java/com/example/docs/attachment/ExampleModAttachments.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Miroma <its.miroma@proton.me>
ekulxam
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All my comments have been resolved.
* main: Developer Guides: revise "Introduction to Fabric" (FabricMC#401) Entity Attributes page (FabricMC#320) Data Attachment Docs (FabricMC#405) Call `setDirty()` Only When Data is Changed (FabricMC#424) Avoid use of first person (FabricMC#423) fix item tooltip, add() -> accept() (FabricMC#422) Fix edit on GitHub links (FabricMC#421)
* plugins: lowercase mixin format sidebar mention first unobfuscated version npm audit fix add tabs plugin Developer Guides: revise "Introduction to Fabric" (FabricMC#401) Entity Attributes page (FabricMC#320) Data Attachment Docs (FabricMC#405) Call `setDirty()` Only When Data is Changed (FabricMC#424) Avoid use of first person (FabricMC#423) fix item tooltip, add() -> accept() (FabricMC#422) Fix edit on GitHub links (FabricMC#421)
* main: add tabs plugin (FabricMC#425) Developer Guides: revise "Introduction to Fabric" (FabricMC#401) Entity Attributes page (FabricMC#320) Data Attachment Docs (FabricMC#405) Call `setDirty()` Only When Data is Changed (FabricMC#424) Avoid use of first person (FabricMC#423) fix item tooltip, add() -> accept() (FabricMC#422) Fix edit on GitHub links (FabricMC#421)
Documents the new Data Attachment API, including info on syncing, persistence, saving and writing, as well as a simplified example of the Facade Pattern discussed by modmuss.
https://deploy-preview-405--nimble-elf-d9d491.netlify.app/develop/data-attachments
Fixes #341