-
Notifications
You must be signed in to change notification settings - Fork 658
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
feat: add documentation for plugins #1412
Conversation
✅ Deploy Preview for tauri-docs-starlight ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Just going to capture a few notes to self here:
Will do a more thorough review ASAP. Would love to use this as a solid reference to do a community/PR push to "develop and show us your Tauri 2.0 plugins" if/when we feel this is all stable enough. |
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
Signed-off-by: Lorenzo Lewis <[email protected]>
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.
Did some formatting on this one, need more input from dev to move further.
I'll try to jump in on this one tomorrow and get the conflicts and structure cleaned up based on the updates and inputs from today. Thanks @lucasfernog 💜 |
@lucasfernog I can see this chunk of documentation going in 1 of 2 ways:
The chunk on Commands I can see really fitting into a Commands/IPC guide and then we split that up a bit by language instead of platform. But I don't know how you want to position this feature and if it's in the spirit of a full-fledged IPC/binding for Swift and Kotlin/Java. The JS IPC is really all handled by Tauri via |
The Kotlin/Swift IPC is only exposed for plugins, so IMO it doesn't make sense to move them out of the plugin documentation. Most application users will just want to leverage existing plugins instead of implementing custom logic for Android and iOS, and those who can do so, will need to write a plugin and go through all that documentation (and hopefully open source it to give back to the community ❤️ ). |
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.
@lucasfernog wrapped up all the formatting and stuff for this one. I have one to-do that I'm going to take care for the CLI reference in the guide, but could you take a look at the other comments?
I think this one is good to go as soon as those are resolved 🥳
--- | ||
|
||
import Stub from '@components/Stub.astro'; | ||
{/* TODO: Add a CLI section */} |
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.
TODO for @lorenzolewis
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.
Finally! (Siri, play Finally by CeCe Peniston)
Thanks @lucasfernog and @simonhyll
No description provided.