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

feat(docs): add more detailed documentation on Transaction Builder #50

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

martinserts
Copy link
Contributor

Description

Updates documentation with more details regarding Transaction Builder.

Motivation and Context

The existing documentation was a bit lacking.

How Has This Been Tested?

The documentation site was tested locally:

$ pnpm install
$ moon tari-docs:build
$ moon tari-docs:serve
tari-docs:build | > tari-docs@0.0.1 build /Users/martinserts/Projects/Tari/tari.js.martins/docusaurus/tari-docs
tari-docs:build | > docusaurus build
tari-docs:build | 
tari-docs:build | [INFO] [en] Creating an optimized production build...
tari-docs:build | [webpackbar] ℹ Compiling Client
tari-docs:build | [webpackbar] ℹ Compiling Server
tari-docs:build | [webpackbar] ✔ Server: Compiled successfully in 7.23s
tari-docs:build | [webpackbar] ✔ Client: Compiled successfully in 10.33s
tari-docs:build | [SUCCESS] Generated static files in "build".
tari-docs:build | [INFO] Use `npm run serve` command to test your build locally.
▪▪▪▪ tari-docs:build (cached, ab3370e3)
▪▪▪▪ tari-docs:serve

> tari-docs@0.0.1 start /Users/martinserts/Projects/Tari/tari.js.martins/docusaurus/tari-docs
> docusaurus start

[INFO] Starting the development server...
[SUCCESS] Docusaurus website is running at: http://localhost:3000/tari.js/

✔ Client
  Compiled successfully in 787.16ms

client (webpack 5.98.0) compiled successfully

What process can a PR reviewer use to test or verify this change?

Start the documentation server as shown above.
Point the browser to: http://localhost:3000/tari.js/

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

@mrnaveira mrnaveira merged commit d51c477 into tari-project:main Feb 20, 2025
4 checks passed
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.

2 participants