Skip to content

Show Razorpay when only paymentProviderConfiguration is set - #2

Open
akadyan-godaddy wants to merge 2 commits into
pchawla-godaddy:vnext-84816from
akadyan-godaddy:feat/razorpay-provider-config-method-bypass
Open

akadyan-godaddy wants to merge 2 commits into
pchawla-godaddy:vnext-84816from
akadyan-godaddy:feat/razorpay-provider-config-method-bypass

Conversation

@akadyan-godaddy

Copy link
Copy Markdown

Summary

  • Mirror the PayPal TEMP bypass for Razorpay: when paymentMethods.razorpay is null/omitted (Airo card-only create) but paymentProviderConfiguration.razorpay.configured === true, still offer Razorpay in the accordion.
  • Synthesize Razorpay method config for Pay now when session method is missing.
  • Ensure the razorpay key is present in the available-methods set when provider config exists (GraphQL may omit null keys).
  • Add tests for null/omitted paymentMethods.razorpay + configured true.

Context

Meant to stack on godaddy#1451 (vnext-84816). Checkout-api discovery can populate paymentProviderConfiguration.razorpay while callers keep explicit card-only paymentMethods, so Razorpay never appeared despite a live Settings config.

Test plan

  • Unit: checkout-razorpay-config tests pass
  • Hosted checkout session with paymentMethods.razorpay: null and paymentProviderConfiguration.razorpay.configured: true shows Razorpay
  • Still hidden when configured is false/missing
  • Pay now renders after selecting Razorpay in that session shape

Made with Cursor

akadyan-godaddy and others added 2 commits September 28, 2026 23:45
Mirror the PayPal TEMP bypass: when callers send explicit card-only
paymentMethods, discovery still fills paymentProviderConfiguration.razorpay
but leaves paymentMethods.razorpay null/omitted. Offer Razorpay from that
configured flag and synthesize method config so Pay now can render.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant