Skip to content

feat: Implement Tokenization for Saved Payment Details (My Account & Checkout)#21129

Merged
Diksha-28 merged 114 commits intodevelopfrom
epic-tokenisation
Apr 3, 2026
Merged

feat: Implement Tokenization for Saved Payment Details (My Account & Checkout)#21129
Diksha-28 merged 114 commits intodevelopfrom
epic-tokenisation

Conversation

@Diksha-28
Copy link
Copy Markdown
Contributor

@Diksha-28 Diksha-28 commented Feb 9, 2026

This PR focuses on implementing tokenisation. The following references are provided to help understand the scope, flow, and implementation details of the items included in this epic.

Epic : CXOPF-4039 Tokenization: Saved Payment Details on Account from PSP payment form during checkout.

Design Document : https://wiki.one.int.sap/wiki/pages/viewpage.action?pageId=5748553978&spaceKey=SAPCCWiki&title=CXOPF-4039%2BTokenization%2BSaved%2BPayment%2BDetails%2Bon%2BAccount%2Bfrom%2Bcheckout.

This PR includes changes for the following Jira tickets :
CXSPA-11820 List saved cards on My Accounts page.
CXSPA-11821 Delete saved cards on My Accounts.
CXSPA-11822 List saved cards on checkout page.
CXSPA-12077 Use saved card to place order.

I've added screen recordings to each Jira ticket for reference, in case you'd like to see how it works in the UI.

@Diksha-28 Diksha-28 requested a review from a team as a code owner February 9, 2026 08:16
@github-actions github-actions Bot marked this pull request as draft February 9, 2026 08:16
@SAP SAP deleted a comment from github-actions Bot Feb 9, 2026
@Diksha-28 Diksha-28 changed the title epic-tokenisation feat : epic-tokenisation Feb 9, 2026
@SAP SAP deleted a comment from github-actions Bot Feb 9, 2026
@SAP SAP deleted a comment from github-actions Bot Feb 9, 2026
@Diksha-28 Diksha-28 changed the title feat : epic-tokenisation feat: epic-tokenisation Feb 9, 2026
@SAP SAP deleted a comment from github-actions Bot Feb 9, 2026
@Diksha-28 Diksha-28 requested a review from Matejk00 February 10, 2026 04:11
Copy link
Copy Markdown
Contributor

@Matejk00 Matejk00 left a comment

Choose a reason for hiding this comment

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

In general looks good, waiting for the final implementation.

@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 07:20
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions Bot marked this pull request as draft April 3, 2026 07:25
@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 07:41
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions Bot marked this pull request as draft April 3, 2026 08:42
@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 08:42
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions Bot marked this pull request as draft April 3, 2026 08:57
@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 08:59
@Diksha-28 Diksha-28 dismissed Matejk00’s stale review April 3, 2026 09:05

Its addressed and re-request review option is not working.

@cypress
Copy link
Copy Markdown

cypress Bot commented Apr 3, 2026

spartacus    Run #52596

Run Properties:  status check passed Passed #52596  •  git commit a5bb2fbb01 ℹ️: Merge 8bf3d06ac4a77c8eb881d6ff2cd658b6c991943d into d7a12f7e9a90ab521c342456d3bc...
Project spartacus
Branch Review epic-tokenisation
Run status status check passed Passed #52596
Run duration 04m 32s
Commit git commit a5bb2fbb01 ℹ️: Merge 8bf3d06ac4a77c8eb881d6ff2cd658b6c991943d into d7a12f7e9a90ab521c342456d3bc...
Committer Diksha-28
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 101
View all changes introduced in this branch ↗︎

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions Bot marked this pull request as draft April 3, 2026 15:22
@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 15:23
@github-actions github-actions Bot marked this pull request as draft April 3, 2026 15:23
@Diksha-28 Diksha-28 marked this pull request as ready for review April 3, 2026 15:23
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@Diksha-28 Diksha-28 merged commit 79f1bb4 into develop Apr 3, 2026
39 of 50 checks passed
@Matejk00
Copy link
Copy Markdown
Contributor

Matejk00 commented Apr 7, 2026

TME and TM status is available under the epic ticket: https://jira.tools.sap/browse/CXOPF-4039

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