Skip to content
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

Docs improvements #347

Merged
merged 15 commits into from Apr 23, 2020
Merged

Docs improvements #347

merged 15 commits into from Apr 23, 2020

Conversation

Stormrose
Copy link
Contributor

Siko91 and I have made some improvements to the Steem-js docs to try and make it easier to onboard new devs. The documentation improvements cover over 40 methods and incorporates our experiences in using those methods in our projects. We have left undocumented methods in place.

Stormrose and others added 10 commits January 23, 2018 01:47
Set explicit coding language tags to all code blocks to improve syntax highlighting.
Plus various other minor spelling improvements.
Also minor formatting and spelling improvements.
Various other fixes
Documentation for: getTrendingTags, getHardforkVersion, getVestingDelegations, getWitnessByAccount
Brief intro for: delegateVestingShares
added documentation for a lot of new methods & improved overall documentation style.
@r351574nc3
Copy link

Can you update the PR a little to trigger rebuild? Looks like Circle failed for some other reason. It just needs to be rerun.

@hernandev
Copy link

👍 for this!

Updates to fix minor issues with whitespace.
@Stormrose
Copy link
Contributor Author

@r351574nc3 I've tried twice. The tests are failing checking the broadcast API. This might be related to the general instability/issues with the API nodes at present.

@r351574nc3
Copy link

This is probably blocked by #349 The fix is #350

@Stormrose
Copy link
Contributor Author

So, I hold off until #350 is merged and then try again?

@r351574nc3
Copy link

Yes. The issues and PRs are now linked. I left a review on #350. I am positive once that is pulled, this can go in as well.

@ety001 ety001 merged commit 79b9646 into steemit:master Apr 23, 2020
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.

None yet

4 participants