Skip to content
This repository was archived by the owner on Feb 23, 2021. It is now read-only.

Conversation

@tanx
Copy link
Contributor

@tanx tanx commented Jun 17, 2019

No description provided.

Copy link
Contributor

@valentinewallace valentinewallace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

Do you want to add this feature to mobile while we're at it? Otherwise follow-up PR is prob fine.

<DetailField name="Status">
{store.selectedTransaction.statusLabel}
</DetailField>
{store.selectedTransaction.preimage ? (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add this to the mobile transaction view as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@valentinewallace good catch! I added it in 6bd1900

@tanx tanx merged commit 9ab1085 into master Jun 18, 2019
@tanx tanx deleted the dev/show-preimage branch June 18, 2019 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants