Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Implement save for later checkbox for Direct Debit #319

Conversation

rinkeshrsys
Copy link
Contributor

@rinkeshrsys rinkeshrsys commented Jul 10, 2020

PR Type

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

Save for later checkbox was not available for Concardis direct debit.

What Is the New Behavior?

Save for later checkbox is visible and working for Concardis direct debit.

Does this PR Introduce a Breaking Change?

[ ] Yes
[X] No

Other Information

Requires Concardis Payment Connector 1.11.x - still unreleased

@shauke shauke requested a review from SGrueber July 11, 2020 11:48
@shauke shauke added the feature New feature or request label Jul 11, 2020
@shauke shauke added this to the 0.22 milestone Jul 11, 2020
@rowiish rowiish self-requested a review July 13, 2020 08:20
Copy link
Contributor

@rowiish rowiish left a comment

Choose a reason for hiding this comment

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

The feature is not complete yet.
The Concardis API requires different directDebitType values for each of the scenarios. Please use SINGLE for one time usage and FIRST when the payment instrument is saved for later re-use.

@SGrueber
Copy link
Collaborator

The highlighted text (see pull request description) should not be displayed this way on the myAccount payment page. Pls remove it, or discuss the IAD.

@SGrueber SGrueber modified the milestones: 0.22, 0.23 Jul 16, 2020
@SGrueber SGrueber added the wait Waiting for something (e.g. new ICM release) label Jul 16, 2020
@shauke shauke modified the milestones: 0.23, 0.24 Aug 19, 2020
@shauke shauke removed the wait Waiting for something (e.g. new ICM release) label Sep 18, 2020
@shauke shauke merged commit a31f16c into intershop:develop Sep 21, 2020
rinkeshrsys added a commit to rinkeshrsys/intershop-pwa that referenced this pull request Sep 22, 2020
rinkeshrsys added a commit to rinkeshrsys/intershop-pwa that referenced this pull request Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants