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.

Sign up for GitHub

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

Jump to bottom

Create "GetPayment" UseCase #93

Merged
merged 2 commits into from
Apr 22, 2022
Merged

Create "GetPayment" UseCase #93

merged 2 commits into from
Apr 22, 2022

Conversation

moiikana
Copy link
Contributor

@gbirke
Copy link
Member

gbirke commented Apr 13, 2022

Please have a look at what LegacyDataProvider does - getPaymentLegacyFormat should do the same (basically moving the code from LegacyDataProvider to the use case.

getPaymentArray should contain all the legacy data (including DE bank data for direct debit).

@moiikana moiikana changed the title Create "GetPayment" UseCase Draft: Create "GetPayment" UseCase Apr 13, 2022
@moiikana moiikana force-pushed the get-payment-usecase branch 5 times, most recently from 7c5d8f6 to a96b9a9 Compare April 20, 2022 16:24
@moiikana moiikana changed the title Draft: Create "GetPayment" UseCase Create "GetPayment" UseCase Apr 20, 2022
@moiikana moiikana merged commit f72371a into refactor-payment-domain Apr 22, 2022
@moiikana moiikana deleted the get-payment-usecase branch April 22, 2022 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gbirke gbirke gbirke approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@moiikana @gbirke