-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Document 2.x API #22
Comments
I've started a I want to hold off on the client/remotes documentation for now, partly b/c I will be adding |
I may also wait on documenting the In any event, I've got a rather terse explanation of the |
I looked at this again, and decided to keep it. Sure, you could just compare And I finally got |
So I took a first pass at using jsdoc-to-markdown to auto-generate an API reference file, based on the JSDoc/TypeScript annotations I've been adding (#25): https://github.com/farmOS/farmOS.js/blob/0e382a58de6dc0a6441482e8cf358b611ff9a4eb/docs/api.md The formatting could be greatly improved, but I'm very happy with the amount of coverage this provides. It took some experimentation to get the npm script right (what does it say that neither JSDoc nor jsdoc-to-markdown provide very good docs?), but now that I've got the hang of it, it should be a snap to regenerate |
All done with the docs, as of c2302d7. Just final touches now for the beta release! 🎉 |
As a matter of expediency, I may punt on fully documenting the 2.x API in favor of farmOS/field-kit#432, so I'm opening this here for tracking.
Checklist
See notes below for items with questions marks (?).
on(?)config(?)Undecided methods (?)
on()
: I may not document this right away, since I'm mostly using this internally, and not sure I want to expose it.isUnsynced()
: I'd like to remove this and simply add a bool to the remote metadata for this instead.config()
: No such method exists yet, but it's a fairly critical piece of functionality I've implemented in FK asconfigDocuments
which might make sense to include here.The text was updated successfully, but these errors were encountered: