Query with REST API #1302
Query with REST API #1302
Conversation
brunopgalvao
added
B0 - Needs Review
brunopgalvao
temporarily deployed
to
n8n-sending
GitHub Actions
Inactive
brunopgalvao
temporarily deployed
to
n8n-receiving
GitHub Actions
Inactive
nhussein11
left a comment
nhussein11
left a comment
There was a problem hiding this comment.
Thanks, left some comments
| curl -s "https://polkadot-asset-hub-public-sidecar.parity-chains.parity.io/accounts/14E5nqKAp3oAJcmzgZhUD2RcptBeUBScxKHgJKU4HPNcKVf3/foreign-asset-balances" | ||
| ``` | ||
|
|
||
| ## Running Sidecar Locally |
There was a problem hiding this comment.
this should be moved to the top, right after ##Prerequisites
| ### Request | ||
|
|
||
| ```bash | ||
| curl -s "https://polkadot-asset-hub-public-sidecar.parity-chains.parity.io/accounts/14E5nqKAp3oAJcmzgZhUD2RcptBeUBScxKHgJKU4HPNcKVf3/balance-info" |
There was a problem hiding this comment.
I think it would be better and cleaner to show something like:
curl -s "https://polkadot-asset-hub-public-sidecar.parity-chains.parity.io/accounts/INSERT_ADDRESS/balance-info"And the same for the rest. If you consider that's valuable to add a concrete address, that could be done in a collapsible element for example
| | `sufficients` | Number of accounts whose existence is paid for by this asset | | ||
| | `status` | Asset status (Live, Frozen, or Destroying) | | ||
|
|
||
| ## Query Block Information |
There was a problem hiding this comment.
I'd move this after foreign assets so it's more consistent; we show all assets-related actions first, then block-related actions.
|
|
||
| For production applications or high-frequency queries, run your own Sidecar instance. | ||
|
|
||
| ### Using npm |
There was a problem hiding this comment.
What about using tabs instead of ###?
|
|
||
| For a complete list of endpoints and parameters, see the [Sidecar API Documentation](https://paritytech.github.io/substrate-api-sidecar/docsv2/){target=\_blank}. | ||
|
|
||
| Common endpoints include: |
There was a problem hiding this comment.
I think this is redundant, considering all of them where explained above
|
|
||
| [workspace] | ||
| members = ["tutorials/polkadot-sdk/parachains/zero-to-hero/runtime"] | ||
| members = ["dummy"] |
| @@ -0,0 +1,2 @@ | |||
| // Placeholder crate for CI workspace requirements. | |||
There was a problem hiding this comment.
I don't think adding these files is a good approach just to satisfy the CI requirements. I'd focus on fixing the CI and investigating why it's failing rather than adding these kinds of files.
📝 Description
Resolves #1205
🔍 Review Preference
Choose one:
🤖 AI-Ready Docs
If content changed, regenerate AI files:
python3 scripts/generate_llms.py✅ Checklist