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: developer tutorial #256

Merged
merged 2 commits into from
Apr 27, 2023
Merged

docs: developer tutorial #256

merged 2 commits into from
Apr 27, 2023

Conversation

homura
Copy link
Contributor

@homura homura commented Apr 23, 2023

What Changed

Resolve #221

Preview at https://github.com/ckb-js/nexus/blob/docs-tutorial/docs/tutorial.md

A tutorial for developer, to show how to create a transaction via FullOwnershipProvider to communicate with Nexus

This PR dependent on

Motivation

A step-by-step tutorial can effectively lower the barrier to entry for developers and give them a sense of accomplishment

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

@homura homura added the documentation Changes only affect the documentation label Apr 23, 2023
@homura homura requested review from zhangyouxin and IronLu233 April 23, 2023 12:01
@homura homura marked this pull request as draft April 23, 2023 12:01
@codecov
Copy link

codecov bot commented Apr 23, 2023

Codecov Report

Merging #256 (b24a7b8) into main (6a43bb7) will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #256      +/-   ##
==========================================
+ Coverage   86.17%   86.25%   +0.08%     
==========================================
  Files          45       47       +2     
  Lines         875      924      +49     
  Branches      114      119       +5     
==========================================
+ Hits          754      797      +43     
- Misses         20       23       +3     
- Partials      101      104       +3     

see 7 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@zhangyouxin
Copy link
Contributor

tested ok
image

docs/tutorial.md Outdated Show resolved Hide resolved
docs/tutorial.md Outdated Show resolved Hide resolved
@homura homura marked this pull request as ready for review April 26, 2023 14:48
@homura homura added released This issue/pull request has been released. and removed released This issue/pull request has been released. labels Apr 26, 2023
@zhangyouxin zhangyouxin merged commit ff8414d into main Apr 27, 2023
@zhangyouxin zhangyouxin deleted the docs-tutorial branch April 27, 2023 05:48
@github-actions
Copy link

🚀 PR was released in v0.0.17 🚀

@github-actions github-actions bot added the released This issue/pull request has been released. label Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes only affect the documentation released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation for ownership-provider
3 participants