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
|
|
||
| [workspace] | ||
| members = ["tutorials/polkadot-sdk/parachains/zero-to-hero/runtime"] | ||
| members = ["dummy"] |
There was a problem hiding this comment.
Without a member the build will break.
| @@ -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.
There was a problem hiding this comment.
Yes agreed, I plan on fixing this once we migrate the tutorial/guide code to the polkadot-cookbook, then we can get rid of this CI completely.
brunopgalvao
force-pushed
the
bruno/query-state-with-rest-api
branch
from
edde452 to
db0c807
Compare
📝 Description
Resolves #1205
🔍 Review Preference
Choose one:
🤖 AI-Ready Docs
If content changed, regenerate AI files:
python3 scripts/generate_llms.py✅ Checklist