Skip to content
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

Merged
merged 12 commits into from
Nov 15, 2022
Merged

explorer-show-recent-package-component #6007

merged 12 commits into from
Nov 15, 2022

Conversation

Jibz1
Copy link
Contributor

@Jibz1 Jibz1 commented Nov 10, 2022

Screenshot 2022-11-12 at 7 38 49 PM

@mystie711
Copy link
Collaborator

The Figma designs have the section lower down on the page,
https://www.figma.com/file/kSD0qXCIDZA4KQVNepA5hR/02-Explorer?node-id=29%3A13465

@Jibz1 Jibz1 changed the title Recent package component explorer-show-recent-package-component Nov 10, 2022
time: <TxTimeType timestamp={timestamp} />,

sender: (
<Link
Copy link
Contributor

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.

Copy link
Contributor Author

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.

timestamp: number | undefined;
};

export function TxTimeType({ timestamp }: Prop) {
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor Author

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

Copy link
Contributor

@Jordan-Mysten Jordan-Mysten left a 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,
Copy link
Contributor

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.

@Jibz1 Jibz1 merged commit 7b60eac into main Nov 15, 2022
@Jibz1 Jibz1 deleted the explorer-recent-modules branch November 15, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants