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: updated tools.md for spvnode #168

Merged

Conversation

edtubbs
Copy link
Collaborator

@edtubbs edtubbs commented Nov 7, 2023

No description provided.

@edtubbs edtubbs force-pushed the 0.1.3-dev-docs_spvnode branch 2 times, most recently from 5d8314c to aa0ec84 Compare November 10, 2023 20:50
Copy link
Member

@xanimo xanimo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have tested all the commands with exception to tpm as i'm on linux atm. everything looks good. i've left a few change requests. lmk if you agree or want to proceed as is.

doc/tools.md Outdated

1. **Header-Only Mode**: This mode is for quickly catching up with the blockchain by downloading only the block headers. It's ideal when resources like bandwidth and storage are limited.

2. **Full Block Validation Mode**: After catching up with the blockchain headers, `spvnode` can switch to this mode to download full blocks for detailed transaction history scanning. This is essential for verifying transactions related to the user's wallet addresses.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would maybe change this to Full Block Mode instead of including validation since validation happens regardless of headers or full blocks.

doc/tools.md Outdated
@@ -2,7 +2,7 @@

## Overview

If you are looking to just explore the functionality of Libdogecoin without building a complicated project yourself, look no further than the CLI tools provided in this repo. The first tool, `such`, is an interactive CLI application that allows you to perform all Essential address and transaction operations with prompts to guide you through the process. The second tool, `sendtx`, handles the process of broadcasting a transaction built using Libdogecoin to eventually push it onto the blockchain.
If you are looking to just explore the functionality of Libdogecoin without building a complicated project yourself, look no further than the CLI tools provided in this repo. The first tool, `such`, is an interactive CLI application that allows you to perform all Essential address and transaction operations with prompts to guide you through the process. The second tool, `sendtx`, handles the process of broadcasting a transaction built using Libdogecoin to eventually push it onto the blockchain. The third tool, `spvnode`, run a Simple Payment Verification (SPV) node for the Dogecoin blockchain. It enables users to interact with the Dogecoin network, verify transactions, and stay in sync with the blockchain without the need for a full node's resource requirements.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would strike without the need for a full node's resource requirements. unless you've done any testing to assert that that claim is true.

doc/tools.md Outdated
## Examples

#### Sync up to the chain tip and stores all headers in `headers.db` (quit once synced):
spvnode scan
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nit with this and all the commands below as having to do with local vs installed. perhaps we should append ./ to ensure the binary being used is the local version to avoid confusion around the situation i encountered yesterday.

@xanimo
Copy link
Member

xanimo commented Nov 17, 2023

one last thing. i'm leaving a reminder that's applicable to #167 and this one and that is that the wallets and headers directories should be removed from both in all instances where that's prepended to the corresponding .db file paths.

@edtubbs edtubbs force-pushed the 0.1.3-dev-docs_spvnode branch 2 times, most recently from 1e66318 to ec0bcb6 Compare November 20, 2023 19:42
Copy link
Member

@xanimo xanimo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@edtubbs edtubbs merged commit 5a924bc into dogecoinfoundation:0.1.3-dev Nov 21, 2023
8 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