sync integrations/makeswift branch w/ canary#2838
Merged
Merged
Conversation
When the sort option is changed in the product filter, the before/after pagination query params are now removed. This prevents stale pagination cursors from being used with the new sort order, which could lead to incorrect results or empty pages. Changes: - Updated Sorting component to use useQueryStates instead of useQueryState - Now manages sort, before, and after params together - Clears before and after params when sort value changes Co-authored-by: Claude <noreply@anthropic.com>
* fix(core): delete duplicate select component * fix: remove console.log
…tion form tooltip (#2814)
* fix(core): persist checkbox modifier in PDP * fix: handle default value for checkbox * fix: revert form changes but keep track of options in params * fix: update changset
…able based on selection (#2818)
* feat(core): add required prop to Label component and update form elements - Introduced a `required` prop to the Label component to indicate mandatory fields. - Updated Input, Textarea, NumberInput, Select, Checkbox, RadioGroup, and other form components to utilize the new `required` prop for better accessibility and user experience. * refactor(core): format required indicator in Label component for improved readability * display optional text for form fields instead of required asterisk * fix linting * add internationalization and translated messages for optional text * remove formatting from da.json file * removed formatting from language files * removed formatting from language files * added changeset * remove 'optional' field from multiple language files * ensure required prop is passed down in form input components * pass required prop to RadioGroupPrimitive.Root * fix linting * removed optional text from checkbox * Pass field.required to CheckboxGroup and Checkbox so that optional text can be displayed
…it, fix e2e tests (#2820)
* Add preventDefault to login form to prevent resetting of form on failed login attempt * added changeset * remove passing of action to form since actions is called manually in onSubmit handler
* chore(ci): BIGCOMMERCE_STORE_HASH secret is now var * chore(ci): BIGCOMMERCE_CHANNEL_ID secret is now var
* refactor: decouple fields from state in DynamicForms * fix: remove options from fields * fix: pass passwordComplexity as an arg * fix: remove fields from return state in contact form * chore: add changeset
* feat(other): LOCAL-1444 delivery translation * chore(core): create translations patch --------- Co-authored-by: bc-svc-local <bc-svc-local@users.noreply.github.com>
* fix(core): remove decorative price element from accessibility tree * fix: update to include accessibility best practices * fix: update changeset * fix: add price label component
* Updated gift certificate balance placeholder * remove placeholders from gift certificates form * add placeholders for coupon code and gift certificate on cart page * added changelog * remove placeholder for coupon code
* chore(catalyst): CATALYST-1297 Run E2E tests on PRs * fix(catalyst): Fix required prop missing on cart shipping form * chore(catalyst): CATALYST-1297 Use test matrix
* chore(ci): pass channel-specific env vars to vercel deploy * chore(ci): harden deploy workflow security replace eval with bash arrays, add timeouts, improve secret handling * chore(ci): add b2b env vars to deploy workflow
* feat(other): LOCAL-1444 delivery translation * chore(core): create translations patch --------- Co-authored-by: bc-svc-local <bc-svc-local@users.noreply.github.com>
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| @@ -1,5 +1,1070 @@ | |||
| # Changelog | |||
|
|
|||
| ## 1.4.1 | |||
Contributor
There was a problem hiding this comment.
Ah you'll probably want to remove this too.
Contributor
Author
There was a problem hiding this comment.
Remove the entire changelog changes?
Contributor
There was a problem hiding this comment.
Yeah, you'll add a new changeset that basically says refer to the Catalyst 1.4.1 changelog which will then generate a new 1.4.1 changelog.
Contributor
There was a problem hiding this comment.
We probably need to add that here: https://github.com/bigcommerce/catalyst/blob/canary/CONTRIBUTING.md#stage-2-sync-and-release-integrationsmakeswift
chanceaclark
approved these changes
Jan 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What/Why?
Syncs
integrations/makeswiftwith latest 1.4.1 changes from canaryTesting
Migration
See CHANGELOG for individual PR migrations