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

Updating model for ACH payment method #47

Merged

Conversation

dcardos
Copy link
Contributor

@dcardos dcardos commented May 13, 2024

Summary

Adding new properties to the model
Updating equality and hash operators
Changing unit tests

dcardos and others added 2 commits May 24, 2024 09:07
* feat: workflow to run unit tests

* fix: authentication issue

* fix: code coverage issue

---------

Co-authored-by: daniloc <danilo.cardoso@adyen.com>
Copy link

sonarcloud bot commented May 24, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@dcardos dcardos merged commit 77f37b3 into feature/Enable-Alternative-Payments May 24, 2024
2 checks passed
@dcardos dcardos deleted the feature/SFI-725-Update-ACH-model branch May 24, 2024 07:51
dcardos added a commit that referenced this pull request May 24, 2024
* SepaDirectDebitDetails and PaymentsResponse classes update (#42)

* chore: updating sepadirectdebitdetails class

* chore: added payment methods in the payments response class

* chore: linting

* chore: linting v2

---------

Co-authored-by: Danilo Cardoso <cardoso.nilo@gmail.com>

* Updating model for ACH payment method (#47)

* feat: updating model for ach payment method

* fix: && position

* fix: lint for test method name

* fix: lint for test method name

* Implementing Unit Test automation (#48)

* feat: workflow to run unit tests

* fix: authentication issue

* fix: code coverage issue

---------

Co-authored-by: daniloc <danilo.cardoso@adyen.com>

---------

Co-authored-by: daniloc <danilo.cardoso@adyen.com>

---------

Co-authored-by: Zenit Shkreli <69572953+zenit2001@users.noreply.github.com>
Co-authored-by: daniloc <danilo.cardoso@adyen.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants