Skip to content

v6 - Improve configuration builders#2755

Open
OscarSpruit wants to merge 3 commits into
mainfrom
chore/improve-configuration-builders
Open

v6 - Improve configuration builders#2755
OscarSpruit wants to merge 3 commits into
mainfrom
chore/improve-configuration-builders

Conversation

@OscarSpruit
Copy link
Copy Markdown
Contributor

Description

This PR improves configuration builders by accepting direct parameters into the extension functions instead of using a lambda. This increases discoverability of the configuration options. Also, the fields are now alphabetically sorted, which makes it easier to find a specific field.

The card extension function now directly accepts parameters. This makes it easier to discover what configuration options are available.

Also, the fields are now alphabetically sorted, which should make it easier to find a specific field.
The threeDS2 extension function now directly accepts parameters. This makes it easier to discover what configuration options are available.

Also, the fields are now alphabetically sorted, which should make it easier to find a specific field.
@OscarSpruit OscarSpruit requested a review from a team as a code owner May 13, 2026 20:12
@OscarSpruit OscarSpruit added the Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes label May 13, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors the configuration API for the Card and 3DS2 components by removing the builder classes and replacing the lambda-based extension functions with versions that use optional parameters and @jvmoverloads. This change simplifies the configuration DSL and improves Java interoperability. I have no feedback to provide.

@github-actions
Copy link
Copy Markdown
Contributor

✅ No public API changes

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
17.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

araratthehero
araratthehero previously approved these changes May 15, 2026
Base automatically changed from chore/card-configuration-alignment to main May 15, 2026 14:36
@OscarSpruit OscarSpruit dismissed araratthehero’s stale review May 15, 2026 14:36

The base branch was changed.

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

Labels

Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes size:small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants