-
Notifications
You must be signed in to change notification settings - Fork 48
Add the react, app-connect, and localizaton packages. #1188
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
base: main
Are you sure you want to change the base?
Conversation
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.
Just a first pass. These grabbed my attention. Will do a 2nd pass once I'm back from vacay
505add7 to
ba52910
Compare
…g generation - Fixed canonicalizeRequest function to prioritize req.path over req.url - This ensures proper signature verification when path is explicitly provided - All tests now pass including signature verification tests Co-authored-by: Amp <[email protected]> Amp-Thread-ID: https://ampcode.com/threads/T-b920e286-dc84-414e-bd6c-5cc01f8216c6
- Added build script to root package.json that builds all packages - Modified test script to run build first, ensuring workspace dependencies are available - This fixes CI failures where react tests failed due to @godaddy/localizations not being built - Resolves workspace dependency resolution issues in GitHub Actions Co-authored-by: Amp <[email protected]> Amp-Thread-ID: https://ampcode.com/threads/T-b920e286-dc84-414e-bd6c-5cc01f8216c6
Amp-Thread-ID: https://ampcode.com/threads/T-99bd6b08-f098-4849-a852-c6b83a9a24b0 Co-authored-by: Amp <[email protected]>
Co-authored-by: Rick Markins <[email protected]>
Co-authored-by: Rick Markins <[email protected]>
Amp-Thread-ID: https://ampcode.com/threads/T-eb0b6a58-0d4f-4ca8-a95f-e7b670beec77 Co-authored-by: Amp <[email protected]>
ba52910 to
3943954
Compare
Amp-Thread-ID: https://ampcode.com/threads/T-eb0b6a58-0d4f-4ca8-a95f-e7b670beec77 Co-authored-by: Amp <[email protected]>
packages/react/src/components/checkout/payment/checkout-buttons/credit-card/square.tsx
Show resolved
Hide resolved
Amp-Thread-ID: https://ampcode.com/threads/T-ba76b785-18f2-4b86-ac29-695004d61704 Co-authored-by: Amp <[email protected]>
- Kept updated README format with comprehensive package table - Added biome-config-godaddy package reference from main - Resolved pnpm-lock.yaml conflicts Amp-Thread-ID: https://ampcode.com/threads/T-6fc81469-d25f-470f-966b-1e45c2474749 Co-authored-by: Amp <[email protected]>
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.
A few comments that are not blocking
| @@ -0,0 +1,23 @@ | |||
| # AGENTS.md - GoDaddy JavaScript Style Guide | |||
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 should just be GoDaddy JavaScript since it's no longer including components that are just for linting and formatting.
| "build": "pnpm run -r build", | ||
| "lint": "pnpm run -r lint", | ||
| "test": "pnpm run -r --parallel test", | ||
| "test": "pnpm run build && pnpm run -r --parallel test", |
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.
the build script could be a pretest hook instead.
Summary
This marks the first stable major release of our GoDaddy JavaScript libraries, representing a significant milestone in our platform development.
A comprehensive React component library for GoDaddy checkout experiences, featuring:
Internationalization support for GoDaddy checkout components:
Essential platform integration tools for GoDaddy app developers:
These packages provide the foundation for building robust, secure, and internationalized applications on the GoDaddy platform.
Changeset
Test Plan