Skip to content

ref: optimized the payment_experience object - #83

Open
Pradeep-kumar1202 wants to merge 1 commit into
mainfrom
payment-experience-optimization
Open

ref: optimized the payment_experience object#83
Pradeep-kumar1202 wants to merge 1 commit into
mainfrom
payment-experience-optimization

Conversation

@Pradeep-kumar1202

@Pradeep-kumar1202 Pradeep-kumar1202 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Type of Change

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

Description

Before : Every time the SDK wanted the experiences for one payment method, it re-scanned the entire sdk-config list from scratch — filter the methods, flatten their sub-types, filter again, flatten the rules, then dedup. The one real caller does this inside the /client response parser.

After : The config is walked once, producing a dictionary from "payment_method:payment_method_type" to its experience list. Answering the question for a method becomes a single O(1) dictionary lookup. The scan cost is paid once per parsed sdk-config instead of once per payment method

  • Doesn't effect web repo independent re-usable func
  • replace the per-lookup sdk-config scan with a build-once payment-experience index

How did you test it?

Impact on Mobile and Web Repositories

Outline steps taken to ensure compatibility with consuming repositories:

  • I tested the submodule changes in the mobile repository.
  • I tested the submodule changes in the web repository.
  • I updated the corresponding documentation in both repositories, if applicable.
  • I confirmed the changes do not introduce regressions in either repository.

Checklist

  • I reviewed submitted code thoroughly.
  • I ensured the changes are compatible with both mobile and web repositories.
  • I communicated potential breaking changes, if any, to the relevant teams.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant