holochain-client-js icon indicating copy to clipboard operation
holochain-client-js copied to clipboard

produce usage documentation

Open Connoropolous opened this issue 4 years ago • 8 comments

to start a discussion, what kind of approach might be reasonable for this?

just write it up in the README? autobuild HTML from typescript types?

Connoropolous avatar Oct 19 '20 16:10 Connoropolous

dialing in a few stakeholders here to collect thoughts... @guillemcordoba @artbrock @maackle @thedavidmeister @JettTech

Connoropolous avatar Oct 19 '20 18:10 Connoropolous

I am willing to do some time on documentation, if I have a proposed approach that has buy-in

Connoropolous avatar Oct 19 '20 18:10 Connoropolous

I'm not sure how instructive generated docs based on types will be on their own, but they'd certainly be helpful.

I guess some actual write-up with examples in the README would be best.

It would be ideal if we already had some documentation on Holochain's conductor API, and then this could be described as a thin layer around that, with some explanation about how to translate between JS and Rust representations of the types. Maybe you could write docs with the assumption that there will also be proper conductor API docs at some point (and an explanation that they don't exist yet).

It also occurs to me that we should probably rename this repo to holochain-conductor-api-js or something, since this is potentially one of many language-specific bindings to the conductor API.

maackle avatar Oct 22 '20 21:10 maackle

Are you thinking these "proper conductor API docs" would be written right into the core codebase as code comments and part of the cargo generated rust docs?

Connoropolous avatar Oct 22 '20 22:10 Connoropolous

@Connoropolous probably, yes. Are you suggesting that we could do the same for this repo? Now that I think of it, that makes sense: my comment about generated docs not being helpful was assuming we'd just have the types, but if there were nice doc comments to go with them, that could be good...haven't given it much thought though.

maackle avatar Oct 26 '20 22:10 maackle

Hey @maackle I've taken a tackle at improving the core documentation of the conductor api docs in https://github.com/holochain/holochain/pull/440/files now...

Can you check it out?

Connoropolous avatar Oct 27 '20 16:10 Connoropolous

@maackle I've done it in preparation for fixing any breaking changes from https://github.com/holochain/holochain/pull/423 and possibly cleaning up some of the internal code as indicated by #24

Connoropolous avatar Oct 27 '20 16:10 Connoropolous

There's this now: https://github.com/holochain/holochain-conductor-api/tree/develop/docs

zippy avatar Mar 04 '21 15:03 zippy

There's the API docs and there are examples in the README.

jost-s avatar Jan 04 '23 21:01 jost-s