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
Fixing broken links in access node docs #1481
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fixes. Just one note on the detail level of the links -- the more specific anchor tags work OK for me with the updated domain.
cmd/access/README.md
Outdated
@@ -40,10 +40,10 @@ At a high level it does the following: | |||
## Processes | |||
|
|||
### Transaction Lifecycle | |||
1. Transactions are received by the access node via the [SendTransaction API call](https://onflow.org/docs/access-api#sendtransaction). | |||
1. Transactions are received by the access node via the [SendTransaction API call](https://docs.onflow.org/access-api/#transactions). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The more detailed anchor links (https://docs.onflow.org/access-api/#sendtransaction
) actually work fine me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, I tried this and it failed but I must have messed up some other part of the URL. Will send an update shortly.
ef3bce9
to
1a6c6a9
Compare
Codecov Report
@@ Coverage Diff @@
## master #1481 +/- ##
==========================================
+ Coverage 55.12% 55.14% +0.01%
==========================================
Files 517 520 +3
Lines 32330 32413 +83
==========================================
+ Hits 17823 17873 +50
- Misses 12116 12143 +27
- Partials 2391 2397 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Some anchors in the original links (SendTransaction, GetTransaction, GetTransactionResult) can only be replaced by pointing to high-level transactions section - there are no method-specific anchors in the new docs.