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

Fixing bug with android pay button #507

Merged
merged 1 commit into from
Apr 3, 2019
Merged

Conversation

eoji
Copy link
Contributor

@eoji eoji commented Apr 2, 2019

What ❓

In this huge PR #427, I added TWO (?!?!) WalletFragments.
I got rid of the second one πŸ™

How Sway?!

I inspected the layout 😭
Screen Shot 2019-04-02 at 6 21 05 PM

I initially thought it was in the toolbar but really it was floating on top ༼ ༎ຢ ෴ ༎ຢ༽ We got around this with the first fragment by setting its height and width to 0dp.

How to QA? πŸ€”

Try to pledge to a project!
The screen should no longer show that ghost button.

Story πŸ“–

Trello Story

See πŸ‘€

(These are Stripe test cards)

@nnekab
Copy link

nnekab commented Apr 2, 2019

giphy

@eoji
Copy link
Contributor Author

eoji commented Apr 3, 2019

Buy with Android Pay

@eoji eoji requested a review from Rcureton April 3, 2019 14:26
Copy link

@ifbarrera ifbarrera left a comment

Choose a reason for hiding this comment

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

Lgtm! πŸ‘

@eoji eoji merged commit 39e711c into master Apr 3, 2019
@eoji eoji deleted the io/android-pay-fragment-bug branch April 3, 2019 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants