-
Notifications
You must be signed in to change notification settings - Fork 235
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
Allow Manual Browser Switch Handling #730
Conversation
b5909d6
to
07d947c
Compare
07d947c
to
9f0b2d2
Compare
…ate BraintreeClient only constructor and onBrowserSwitchResult with callback method.
Signed-off-by: Jax DesMarais-Leder <[email protected]>
91ba131
to
593a145
Compare
@@ -32,7 +33,7 @@ public class LocalPaymentClient { | |||
/** | |||
* Create a new instance of {@link LocalPaymentClient} from within an Activity using a {@link BraintreeClient}. | |||
* | |||
* @param activity a {@link FragmentActivity} | |||
* @param activity a {@link FragmentActivity} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The spacing for this looks different then the braintreeClient
right below. Is this intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's the auto formatter in Android studio. We would need to run it on each file for 100% consistency.
Co-authored-by: Jax DesMarais-Leder <[email protected]>
Co-authored-by: Jax DesMarais-Leder <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👻
Summary of changes
Checklist
Authors