-
Notifications
You must be signed in to change notification settings - Fork 297
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
[DDW-521] Display Transaction Metadata #2338
[DDW-521] Display Transaction Metadata #2338
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.
@DominikGuzei I don't see the metadata which is visible on the Explorer 😢
TX: https://explorer.cardano-testnet.iohkdev.io/en/transaction?id=4dc5093d26fdd8d82ba30615eda72d96b80ed28d3750d916ab1d3ab6fee4b4b3
@nikolaglumac these issues should be fixed now 👍 |
@DominikGuzei please work with @darko-mijic on getting the final text copy and Japanese translations. We need this merged by the end of the day tomorrow 🙏 PS. @darko-mijic this PR is important for Fund3! |
….com:input-output-hk/daedalus into feature/ddw-521-display-transaction-metadata
@DominikGuzei Here are a few issues on the latest build (16293):
|
@gnpf the second issue is fixed 👍 |
@gnpf @a-rukin @ManusMcCole @mioriohk please test this one 🙏 |
@gnpf I have fixed this ☝️ |
@DominikGuzei this padding feels a bit smaller than 20px |
I have checked this. This is an illusion. It is all OK! |
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.
Nice work @DominikGuzei 👍
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.
LGTM
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.
Looks good 🎉
Great work everyone 🎉 |
This PR adds support for rendering transaction metadata:
Todos
Screenshots
Rendering nested maps:
Metadata maps can contain any metadata values (int, string, bytes, lists, maps) in keys AND values – simple scalars (int, string, bytes) are stringified and simply used as string-based keys. Compound types like lists and maps are not valid JSON keys, so they are saved under "key" property of the map (with a corresponding "value" property).
Testing Checklist
Test Cases
Scenario1 - Verify link for transaction metadata is present
Scenario 2 - Verify Link works correctly and expands to show content
Scenario 3 - Verify that all transaction render correctly regardless of metadata
Wallet with metadata for testing
barely endorse vote squirrel rocket barrel meadow soft wheel cup income hamster work always country speed rocket person core protect evolve radio claim dish
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board