feat: add PR-wise changelog#2833
Conversation
|
React Doctor found 5 issues in 3 files · 5 warnings · score 81 / 100 (Needs work) · vs 5 warnings
|
|
|
||
| <div className="space-y-14"> | ||
| {dateGroup.entries.map(({ story, leadPullRequest, pullRequests }) => { | ||
| const relatedPullRequests = pullRequests |
There was a problem hiding this comment.
React Doctor · react-doctor/js-combine-iterations (warning)
This loops over your list twice because .filter().map() makes two passes, so do it in one pass with .reduce() or a for...of loop
Fix → Combine .map().filter() style chains into one pass with .reduce() or a for...of loop, so you only loop over the list once
| }; | ||
|
|
||
| function formatPullRequestDate(date: string) { | ||
| return new Intl.DateTimeFormat("en-US", { |
There was a problem hiding this comment.
React Doctor · react-doctor/js-hoist-intl (warning)
This is slow because new Intl.DateTimeFormat() rebuilds on every call inside a function, so move it to the top of the file, or wrap it in useMemo
Fix → Move new Intl.NumberFormat(...) to the top of the file or wrap it in useMemo. Building one is slow, so don't redo it on every call
What changed
/changelogroute based on the Magic UI changelog templateWhy
A public, product-focused changelog gives landing-page visitors a clear history of shipped work while preserving links to every underlying pull request for deeper technical detail.
Impact
Visitors can browse the project history in ten readable pages using the same colors, typography, navigation, and interaction language as the landing page. Every included story and source link is derived from real repository data.
Validation
npm run buildinfrontend/src/landing/changelogand/changelog?page=10return HTTP 200 locally