Skip to content

feat: sdk config api integration#1577

Merged
ArushKapoorJuspay merged 15 commits into
mainfrom
feat-sdk-config-api
Jun 12, 2026
Merged

feat: sdk config api integration#1577
ArushKapoorJuspay merged 15 commits into
mainfrom
feat-sdk-config-api

Conversation

@Shivam25092001

@Shivam25092001 Shivam25092001 commented May 20, 2026

Copy link
Copy Markdown
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

let options = {
{
    clientSecret,
    appearance: {
      labels: "floating",
      // theme: "brutal",
    },
    preloadSDKWithParams: params,
  };
}

<HyperElements hyper={hyperPromise} options={options}>
          <CheckoutForm />
        </HyperElements>
{
    paymentMethodsList: ... ,
    customerMethodsList: ... ,
    sessionTokens: ...,
    sdkConfigs: ...
}

How did you test it? [TESTING TO BE DONE AFTER BACKEND DEPLOYMENT]

Added a console.log in DynamicFields.res file to print the sdkConfigs state.

  let sdkConfig = Recoil.useRecoilValueFromAtom(sdkConfigs)
  Console.log2("-- DynamicFields sdkConfigs -- ", sdkConfig)
image
  • Tried passing sdkAuthorization in <HyperElements hyper={hyperPromise} options={{sdkAuthorization}}>, profile_id retrieved successfully and sdk-configs api fetched properly.
  • Tried passing profile_id in window.Hyper({ publishableKey, profileId }), profile_id retrieved successfully and sdk-configs api fetched properly.
  • Tried passing older cs without attached profiled_id, got integration error in console.
  • Also tested by passing preloadSDKWithParams.

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

@semanticdiff-com

semanticdiff-com Bot commented May 20, 2026

Copy link
Copy Markdown

Review changes with  SemanticDiff

@github-actions

Copy link
Copy Markdown
Contributor

🚫 Missing Linked Issue

Hi 👋 This pull request does not appear to be linked to any open issue yet.

Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically.

✔️ How to fix this

  • Add a keyword like Fixes #123 or Closes #456 to your PR description or a commit message.
  • Or link it manually using the "Linked issues" panel in the PR sidebar.

Tip: You can link multiple issues.
🚫 Note: If only one issue is linked, it must be open for this check to pass.

Once linked, this check will pass automatically on your next push or when you re-run the workflow.

Thanks for helping maintainers! 🙌

@github-actions github-actions Bot added the S-conventions-not-followed Some Github conventions not followed label May 20, 2026
Comment thread src/hyper-loader/Elements.res
Comment thread src/Utilities/APIHelpers/APIUtils.res Outdated
Comment thread src/Payments/PreMountLoader.res
@aritro2002

aritro2002 commented May 20, 2026

Copy link
Copy Markdown
Contributor

please add more description and test cases images and video. Link issue.

Comment thread src/hyper-loader/UpdateIntentHelpersNew.res Outdated
Comment thread src/Utilities/PaymentHelpers.res Outdated
@github-actions

Copy link
Copy Markdown
Contributor

🚫 Missing Linked Issue

Hi 👋 This pull request does not appear to be linked to any open issue yet.

Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically.

✔️ How to fix this

  • Add a keyword like Fixes #123 or Closes #456 to your PR description or a commit message.
  • Or link it manually using the "Linked issues" panel in the PR sidebar.

Tip: You can link multiple issues.
🚫 Note: If only one issue is linked, it must be open for this check to pass.

Once linked, this check will pass automatically on your next push or when you re-run the workflow.

Thanks for helping maintainers! 🙌

@aritro2002

Copy link
Copy Markdown
Contributor

also please check the PR title, and other S conventions which are not followed.

@github-actions

Copy link
Copy Markdown
Contributor

🚫 Missing Linked Issue

Hi 👋 This pull request does not appear to be linked to any open issue yet.

Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically.

✔️ How to fix this

  • Add a keyword like Fixes #123 or Closes #456 to your PR description or a commit message.
  • Or link it manually using the "Linked issues" panel in the PR sidebar.

Tip: You can link multiple issues.
🚫 Note: If only one issue is linked, it must be open for this check to pass.

Once linked, this check will pass automatically on your next push or when you re-run the workflow.

Thanks for helping maintainers! 🙌

@Shivam25092001 Shivam25092001 changed the base branch from feat-profileId-cs to main May 20, 2026 17:42
@github-actions

Copy link
Copy Markdown
Contributor

🚫 Missing Linked Issue

Hi 👋 This pull request does not appear to be linked to any open issue yet.

Linking your PR to an issue helps keep the project tidy and ensures the issue is closed automatically.

✔️ How to fix this

  • Add a keyword like Fixes #123 or Closes #456 to your PR description or a commit message.
  • Or link it manually using the "Linked issues" panel in the PR sidebar.

Tip: You can link multiple issues.
🚫 Note: If only one issue is linked, it must be open for this check to pass.

Once linked, this check will pass automatically on your next push or when you re-run the workflow.

Thanks for helping maintainers! 🙌

Comment thread src/hyper-loader/UpdateIntentHelpersNew.res Outdated
Comment thread src/Utilities/APIHelpers/APIUtils.res Outdated
Comment thread src/Utilities/APIHelpers/APIUtils.res Outdated
Comment thread src/hyper-loader/UpdateIntentHelpersNew.res Outdated
@ArushKapoorJuspay ArushKapoorJuspay merged commit 7f78e89 into main Jun 12, 2026
12 of 14 checks passed
@ArushKapoorJuspay ArushKapoorJuspay deleted the feat-sdk-config-api branch June 12, 2026 14:51
@github-actions github-actions Bot added the Closed Label will be automatically added when the PR will get merged to main label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Closed Label will be automatically added when the PR will get merged to main Ready for Review PR with label Ready for Review should only be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate 'sdk-configs' api for superposition-raw-configs

7 participants