-
Notifications
You must be signed in to change notification settings - Fork 60
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
Fix: sort SKU variations #2367
base: main
Are you sure you want to change the base?
Fix: sort SKU variations #2367
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Nice! Thank you for the PR.
Do you have a store so that we can validate this change with the version generated from the codesandbox?
Also, we need to validate this rule with FastStore team since are cases we should respect the catalog order.
Finally, the build is not passing. Can you take a look?
) | ||
|
||
const isNumeric = (value) => { | ||
return !isNaN(value - parseFloat(value)) |
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.
This looks confusing. Why do we need this to check if it is a number?
@wender Hello! Thanks for opening this PR. I noticed it was opened some time ago. For more context, was it related to a specific store issue? |
What's the purpose of this pull request?
Currently, the SKU variations are not sorted
How it works?
It now checks if the value is a number or string and sort it alphabetically or numerically ascendant
How to test it?
Open a PDP with any of the SKU variations, number or text
Starters Deploy Preview
References