-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat: show receipt time on transaction page #1227
Conversation
@dragnoir is attempting to deploy a commit to the Hiro Systems Team on Vercel. A member of the Team first needs to authorize it. |
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.
Thank you @andresgalante, margin left 16px removed |
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.
Codecov Report
@@ Coverage Diff @@
## main #1227 +/- ##
==========================================
- Coverage 23.05% 6.14% -16.92%
==========================================
Files 453 382 -71
Lines 6635 5775 -860
Branches 1695 1435 -260
==========================================
- Hits 1530 355 -1175
- Misses 4933 5244 +311
- Partials 172 176 +4
|
Awesome @dragnoir! Please fix linting so that we can proceed with shipping it. |
Also, wondering if you could add a snapshot test for src/app/txid/[txid]/TxDetails/Broadcasted.tsx. This would be amazing! |
59dbfca
to
e6c587f
Compare
e6c587f
to
770d56e
Compare
770d56e
to
1eeab5e
Compare
🎉 This PR is included in version 1.121.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Close #1030
On the transaction details page, for pending transactions (mempool), I added a new component for receipt time labeled "Broadcasted"