View XDR stream + update Trezor deps #1178
Conversation
|
Preview is available here: |
src/app/(sidebar)/xdr/view/page.tsx
Outdated
| json={parseToLosslessJson(xdrJsonDecoded.jsonString)} | ||
| xdr={xdr.blob} | ||
| /> | ||
| {xdrJsonDecoded?.jsonArray?.map((j, index) => ( |
There was a problem hiding this comment.
Should it be array of Json objects instead of array of PrettyJsonTransaction?
I think it would be a bit more user friendly, currently I have
{
"function_v0": {...}
}
{
"function_v0":{...}
}
Which is not a valid JSON (so you can't just pass it to any of JSON parsers). Instead, maybe we can out a valid JSON array of
[
{
"function_v0": {...}
},
{
"function_v0":{...}
}
]
WDYT?
There was a problem hiding this comment.
Thanks for pointing this out. I'll update it to display it as a proper array of JSON objects.
Note: To get the correct JSON, you should use the "Copy JSON" button (uses direct response without any modification). We have extra elements in the UI that will make the JSON invalid if you select and copy it.
|
Preview is available here: |
| <div className="PrettyJson PrettyJson__array"> | ||
| <div className="PrettyJson__inline"> | ||
| <span className="PrettyJson__bracket">[</span> | ||
| <span className="PrettyJson__expandSize">{`${jsonArray.length} items`}</span> | ||
| </div> | ||
| {jsonArray.map((j, index) => ( | ||
| <> | ||
| <PrettyJsonTransaction | ||
| // Using index here because we can't get something unique from the JSON | ||
| key={`pretty-json-${index}`} | ||
| json={j} | ||
| xdr={xdr} | ||
| /> | ||
| </> | ||
| ))} | ||
| <span className="PrettyJson__bracket">]</span> | ||
| </div> |
There was a problem hiding this comment.
Handling the array styling manually here because it's the only place where we have it. If we need it someplace else, we'll create a component for it.
XDR stream example
Type
ScSpecEntryXDR