Sync recent changes from canary into integrations/b2b-buyer-portal
#2604
Conversation
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
Use an onSubmit listener and move most of the logic outside the product-details-form component. The ideal solution here is to patch the form action at the root of the component tree, but that's proven complicated at this point. Will try to amend it in the future.
* feat: Add support for Buyer Portal in Catalyst * refactor: remove unnecessary useState/useEffects + simplify useB2B* logic * refactor: simplify use of env. variables * refactor: Simplify integration points with product-details-form Use an onSubmit listener and move most of the logic outside the product-details-form component. The ideal solution here is to patch the form action at the root of the component tree, but that's proven complicated at this point. Will try to amend it in the future. * fix: cart sync * fix: deprecating sync-cart * feat: Add support for Buyer Portal in Catalyst * refactor: remove unnecessary useState/useEffects + simplify useB2B* logic * refactor: simplify use of env. variables * refactor: Simplify integration points with product-details-form Use an onSubmit listener and move most of the logic outside the product-details-form component. The ideal solution here is to patch the form action at the root of the component tree, but that's proven complicated at this point. Will try to amend it in the future. * fix: changes according to review * fix: changes according to review * fix: improving cartId prop type --------- Co-authored-by: Ignacio Catalina <[email protected]>
feat: Add support for Buyer Portal in Catalyst
feat: Add support for Buyer Portal in Catalyst
…rtal (#2504) * Version Packages (`canary`) (#2452) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Bump next to latest canary (#2465) * docs: how to verify new merge base established when syncing (#2451) Improve explanation around the importance of establishing a new merge base every time `canary` is merged into `integrations/makeswift` * chore(core): remove runtime edge declarations (#2464) * Update translations (#2468) * feat(other): LOCAL-1444 delivery translation * chore(core): create translations patch --------- Co-authored-by: bc-svc-local <[email protected]> Co-authored-by: Jorge Moya <[email protected]> * Use min and max purchase quantity to set bounds of quantity number input (#2474) * chore: update ci to include @bigcommerce/catalyst (#2463) * chore: update deps mostly related to packages/cli (#2461) * chore: use @commander-js/extra-typings as devDep (#2462) * feat: render scripts from Script Manager (#2477) * chore: hoist eslint plugins for ide support (#2460) * feat: recursively copy cwd into temp folder (#2479) * feat(cli): scaffold deploy command and generate bundle zip (#2481) * feat(cli): scaffold deploy command and generate bundle zip * feat: remove archiver and only use adm-zip * feat: add extra checks for build files * refactor: move checks around * fix: use .bigcommerce/dist as build path * refactor: split mkTempDir to own lib file and update tests * fix: remove file * fix: set default value for prefix * fix: remove extra-typings * feat: install and build all deps required to bundle catalyst (#2486) * chore: bump opennextjs-cloudflare version (#2487) * feat(cli): generate upload signature and upload bundle.zip (#2484) * feat(cli): generate upload signature and upload bundle.zip * feat: read from env variables * feat: use zod to validate response * refactor: parse * feat: read api host from env * refactor: infer option types in deploy command action (#2490) * feat: copy templates from cli to catalyst build tmpdir (#2489) * Version Packages (`canary`) (#2466) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Nathan Booker <[email protected]> Co-authored-by: Matthew Volk <[email protected]> Co-authored-by: Jorge Moya <[email protected]> Co-authored-by: bc-svc-local <[email protected]> Co-authored-by: bc-svc-local <[email protected]>
* feat: adding a safeguard to the B2B API host * feat: adding a warning when app starts instead of login * feat: getApiHostname and error throw on dev * fix: adding chalk to core package * fix: changes according to review * chore: chalk cleanup
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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?
Regular sync to make cutting a release easier to reason about (smaller diff) next time we release.
Testing
Manually reviewed the diff, all tasks (
lint,typecheck,test) pass, manual testing also successful:sync-integrations-b2b-testing.mov
Migration
N/A
Jira: CATALYST-1492