-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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
explorer-show-recent-package-component #6007
Conversation
Jibz1
commented
Nov 10, 2022
•
edited
Loading
edited
The Figma designs have the section lower down on the page, |
apps/explorer/src/components/recent-packages-card/RecentPackagesCard.tsx
Show resolved
Hide resolved
apps/explorer/src/components/recent-packages-card/RecentPackagesCard.tsx
Outdated
Show resolved
Hide resolved
apps/explorer/src/components/recent-packages-card/RecentPackagesCard.tsx
Outdated
Show resolved
Hide resolved
time: <TxTimeType timestamp={timestamp} />, | ||
|
||
sender: ( | ||
<Link |
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.
I'd be open to a new InternalLink
(probably need a better name) that took a type
like address, object, etc and linked there, and also managed consistent truncation. Basically LongText
but rebuilt on new UI components.
You don't need to do this now, we can create a backlog item for 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.
Agreed, we can just build that out later.
apps/explorer/src/components/recent-packages-card/RecentPackagesCard.tsx
Outdated
Show resolved
Hide resolved
timestamp: number | undefined; | ||
}; | ||
|
||
export function TxTimeType({ timestamp }: Prop) { |
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.
We could also make this a base UI component and port over the time utilities.
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.
Agreed. I wanted to get the high-priority done first.
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.
We can combine this component with the DateCard component on the other PR
067f611
to
84fd4c3
Compare
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.
Approving to unblock, but please address the comments before merging
return recentPublishMod.data; | ||
}, | ||
{ | ||
staleTime: TRANSACTION_STALE_TIME, |
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.
Do we also want auto-refreshing here? cc @mystie711 curious what your thoughts are.