-
Notifications
You must be signed in to change notification settings - Fork 619
[TOOL-3368] Dashboard: Improve page load perf of explore page and other few others #6214
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6214 +/- ##
=======================================
Coverage 56.72% 56.72%
=======================================
Files 1161 1161
Lines 64289 64289
Branches 5196 5196
=======================================
Hits 36468 36468
Misses 27095 27095
Partials 726 726
*This pull request uses carry forward flags. Click here to find out more. |
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
size-limit report 📦
|
| contractId: contractId, | ||
| }); | ||
|
|
||
| const sortedVersions = allVersions.toSorted((a, b) => { |
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.
fwiw this toSorted was throwing an error for me locally. I had to temporarily set it to [...allVersions].sort() to get it working locally. Not sure if it's an issue.
Merge activity
|
…er few others (#6214) <!-- start pr-codex --> ## PR-Codex overview This PR focuses on updating the contract fetching logic in the `dashboard` application to improve the retrieval of the latest published contracts. It replaces older functions with new ones that specifically fetch the latest contract versions. ### Detailed summary - Added `export const dynamic = "force-static";` to `page.tsx` files. - Replaced `getPublishedContractsWithPublisherMapping` with `getLatestPublishedContractsWithPublisherMapping` in multiple files. - Introduced `getLatestPublishedContractsWithPublisherMapping` function to fetch the latest contract version. - Updated `fetchPublishedContractVersions` to use `getSortedPublishedContractVersions`. - Created `fetchLatestPublishedContractVersion` for fetching the latest version details. - Modified `ContractCard` to use `ClientOnly` for rendering the `ContractPublisher`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
4815a4e to
bc42c2f
Compare

PR-Codex overview
This PR focuses on enhancing the contract management system by updating the methods used to fetch published contracts. It introduces a new function to get the latest published contracts and modifies several components to utilize this new functionality.
Detailed summary
export const dynamic = "force-static";topage.tsxfiles.getPublishedContractsWithPublisherMappingwithgetLatestPublishedContractsWithPublisherMappingacross multiple files.getLatestPublishedContractsWithPublisherMappingfunction.fetchPublishedContractVersionsfunction to includegetSortedPublishedContractVersions.