Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: d8c79c0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Bundle Size ReportComparing against baseline from No bundle size changes detected. |
Unlighthouse Performance Comparison — VercelComparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores. Summary ScoreAggregate score across all categories as reported by Unlighthouse.
Category Scores
Core Web Vitals
|
|
Also, can we clean up the commits a bit here. Looks like most of them are just changing wording? You can either amend or squash as needed. |
Brand PLPs were defaulting to relevance, which is intended for search results. Pin the default to featured so unranked brand pages match the rest of the catalog.
parthshahp
left a comment
parthshahp
left a comment
There was a problem hiding this comment.
Looks good, approved! Also just to double check: "featured" is meant to be the default, correct?
3 participants
JIRA
TRAC-279
What / Why
Brand pages weren't showing products in the order the merchant set in the
Control Panel.
Catalyst wasn't telling the backend which sort to use when a shopper landed
on a brand page without picking one. The backend then fell back to its own
default, which doesn't honor the merchant's chosen order. Asking for
"featured" explicitly fixes this and matches the sort dropdown's default
selection.
Rollout / Rollback
No flag. Backwards compatible: the new default only applies when no
?sort=param is set, so any existing URL with a sort param is unaffected. Rollback is reverting the single line.Testing
Tested in catalyst store. The first screenshot is featured items all with sort = 0. The second screenshot is featured items where Spray Bottle was assigned 1, Snake Plant was assigned -1, and ZZ Plant was left at 0.
🤖 Generated with Claude Code