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

PW-1705 #323

Merged
merged 5 commits into from
Mar 27, 2020
Merged

PW-1705 #323

merged 5 commits into from
Mar 27, 2020

Conversation

andrei-dediu
Copy link
Contributor

Added the store field to the PaymentsRequest and PaymentMethodsRequest for the Checkout API.

Description

Added the store property field to the PaymentsRequest and PaymentMethodsRequest manually as it was requested for the ticket. The OpenAPI models have not yet been updated with these fields so we can not generate them yet.

Tested scenarios

None.

Fixed issue:

PW-1705

Added the store field to the PaymentsRequest and PaymentMethodsRequest for the Checkout API.
AlexandrosMor
AlexandrosMor previously approved these changes Mar 25, 2020
@coveralls
Copy link

coveralls commented Mar 25, 2020

Coverage Status

Coverage decreased (-0.006%) to 24.54% when pulling 1ab62fd on feature/PW-1705 into 8d6b2cc on develop.

@@ -87,6 +87,9 @@
@SerializedName("threeDSAuthenticationOnly")
private Boolean threeDSAuthenticationOnly = null;

@SerializedName("store")
private String store = null;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update equals() and toString() methods to add this field

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch, updated.
Also, the applicationInfo serialised annotation was mispelled, updated that as well.

Updated toString and hash of PaymentsRequest.
Fixed mispelled property name in PaymentMethodRequest.
martinsrenato
martinsrenato previously approved these changes Mar 25, 2020
rkewlani
rkewlani previously approved these changes Mar 25, 2020
AlexandrosMor
AlexandrosMor previously approved these changes Mar 27, 2020
@martinsrenato martinsrenato self-requested a review March 27, 2020 13:33
@andrei-dediu andrei-dediu merged commit 64a5c23 into develop Mar 27, 2020
@rikterbeek rikterbeek mentioned this pull request Apr 28, 2020
@wboereboom wboereboom deleted the feature/PW-1705 branch October 14, 2022 06:21
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.

None yet

6 participants