add checkoutOption to GooglePayRequest#1514
Conversation
saralvasquez
left a comment
There was a problem hiding this comment.
Could you please include an entry in the CHANGELOG under the unreleased header detailing what changed in this PR? Look at other entries for an example
saralvasquez
left a comment
There was a problem hiding this comment.
Looks really good but I have a couple small comments
| Settings.isGooglePayShippingAddressRequired(activity)); | ||
| googlePayRequest.setShippingAddressParameters(new GooglePayShippingAddressParameters(Settings.getGooglePayAllowedCountriesForShipping(requireContext()))); | ||
|
|
||
| googlePayRequest.setCheckoutOption(GooglePayCheckoutOption.COMPLETE_IMMEDIATE_PURCHASE); |
There was a problem hiding this comment.
| googlePayRequest.setCheckoutOption(GooglePayCheckoutOption.COMPLETE_IMMEDIATE_PURCHASE); | |
| googlePayRequest.setCheckoutOption(GooglePayCheckoutOption.COMPLETE_IMMEDIATE_PURCHASE); | |
|
@fatlinesofcode - are you able to address the feedback above? If so, we would be happy to re-review this PR. If we do not hear back we will go ahead and close this. |
|
updated now @jaxdesmarais |
saralvasquez
left a comment
There was a problem hiding this comment.
This semi colon is causing lint issues. Once it's removed this should be good to go in
|
Hi! Sorry this has been sitting for so long. We just got confirmation that this can be merged despite the security check permission issues. However, we don't have permissions on this fork to resolve some merge conflicts so could you please resolve those on your end? We'll go ahead and do one last check after that and then get this merged in |
# Conflicts: # CHANGELOG.md # GooglePay/src/main/java/com/braintreepayments/api/googlepay/GooglePayRequest.kt # GooglePay/src/test/java/com/braintreepayments/api/googlepay/GooglePayRequestUnitTest.kt
thats done now |
saralvasquez
left a comment
There was a problem hiding this comment.
This needs a couple small changes to the CHANGELOG entry
Co-authored-by: Sara Vasquez <98496950+saralvasquez@users.noreply.github.com> Signed-off-by: Philip Andrews <phil@frontside.com.au>
Co-authored-by: Sara Vasquez <98496950+saralvasquez@users.noreply.github.com> Signed-off-by: Philip Andrews <phil@frontside.com.au>
Co-authored-by: Sara Vasquez <98496950+saralvasquez@users.noreply.github.com> Signed-off-by: Philip Andrews <phil@frontside.com.au>
|
There's more conflicts that we can't resolve from our side. Please resolve those. We'd also be happy to manage all of this if you change the permissions on this branch so that we can make commits |
# Conflicts: # CHANGELOG.md
|
hi @saralvasquez those conflicts are resolved now. i dont believe i can give you access to our organisation's branch |
| */ | ||
| DEFAULT("DEFAULT"), | ||
|
|
||
| CONTINUE_TO_REVIEW("CONTINUE_TO_REVIEW"), |
There was a problem hiding this comment.
This one is missing docstrings
| * GooglePay | ||
| * Add optional `DisplayItems` in GooglePay request | ||
| * Add support for `checkoutOption` to `GooglePayRequest` TransactionInfo params | ||
|
|
Summary of changes
add support checkoutOption param to GooglePayRequest
Checklist
Authors
@fatlinesofcode