Skip to content

Comments

feat(checkout): add CCAvenue payment provider and cookie-session retu…#1291

Open
asaini-godaddy wants to merge 2 commits intogodaddy:mainfrom
asaini-godaddy:vnext-710674
Open

feat(checkout): add CCAvenue payment provider and cookie-session retu…#1291
asaini-godaddy wants to merge 2 commits intogodaddy:mainfrom
asaini-godaddy:vnext-710674

Conversation

@asaini-godaddy
Copy link

…rn flow

  • Add CCAvenue checkout button, icon, and return provider
  • Wrap with CCAvenueReturnProvider only when ccavenueConfig.accessCodeId is set
  • Use session from context (cookie) on CCAvenue return; remove sessionStorage token flow
  • Register CCAvenue in lazy-payment-loader, payment-form, types, and GraphQL
  • Update conditional-providers and use-confirm-checkout for CCAvenue
  • Add ccavenue localizations across locales
  • Update nextjs example and store actions

Summary

Changeset

  • Changeset added (docs)

Test Plan

…rn flow

- Add CCAvenue checkout button, icon, and return provider
- Wrap with CCAvenueReturnProvider only when ccavenueConfig.accessCodeId is set
- Use session from context (cookie) on CCAvenue return; remove sessionStorage token flow
- Register CCAvenue in lazy-payment-loader, payment-form, types, and GraphQL
- Update conditional-providers and use-confirm-checkout for CCAvenue
- Add ccavenue localizations across locales
- Update nextjs example and store actions

Co-authored-by: Cursor <cursoragent@cursor.com>
@asaini-godaddy asaini-godaddy requested a review from a team as a code owner February 18, 2026 20:34
@changeset-bot
Copy link

changeset-bot bot commented Feb 18, 2026

⚠️ No Changeset found

Latest commit: 086c65f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

__schema: {
queryType: {
name: 'Query',
"name": "checkout-api",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

run pnpm lint:fix in packages/react to clean this up.

- Pass ccavenueConfig from env in Next.js example (accessCodeId, redirectURL)
- CCAvenue button: user-friendly message for AUTHORIZATION_FAILED, remove debug logs
- CCAvenue return: use useConfirmCheckout.mutateAsync for redirect and error handling
- Remove Record typecasting for payment.methods.ccavenue and descriptions in payment-form

Co-authored-by: Cursor <cursoragent@cursor.com>
? {
accessCodeId:
process.env.NEXT_PUBLIC_CCAVENUE_ACCESS_CODE_ID,
redirectURL: process.env.NEXT_PUBLIC_CCAVENUE_REDIRECT_URL,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should be setting the redirectURL in the payment component itself how we load payment method js files for example. The consumer of checkout shouldn't have to know what URL ccavanue needs to use to create their checkout.

const squareCDN =
    apiHost && !apiHost.includes('test') && !apiHost.includes('dev')
      ? 'https://web.squarecdn.com/v1/square.js'
      : 'https://sandbox.web.squarecdn.com/v1/square.js';

type='button'
size='lg'
className={cn('w-full')}
disabled={isPaymentDisabled || isBusy}
Copy link
Collaborator

@pbennett1-godaddy pbennett1-godaddy Feb 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isPaymentDisabled is already part of isBusy

Suggested change
disabled={isPaymentDisabled || isBusy}
disabled={isBusy}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants