-
Notifications
You must be signed in to change notification settings - Fork 292
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
fix: donation UI visibility issue #1229
Conversation
- chore: update target SDK to 34 Signed-off-by: Amr Hossam <[email protected]>
Signed-off-by: Amr Hossam <[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.
Can you split the two commits into:
- target SDK
- donation UI visibility?
YOC |
This reverts commit 31b34f0.
This reverts commit bee5f07.
Signed-off-by: Amr Hossam <[email protected]>
This reverts commit 933f688.
Signed-off-by: Amr Hossam <[email protected]>
This pull request addresses the visibility issues with the donation UI. I will submit a separate pull request for the SDK upgrade, which will involve additional changes. |
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.
looks great! 👍
fixes: #1227
Video
2024-08-27.20-11-14.mp4
Apply the
AndroidStyle.xml
style template to your code in Android Studio.Run the unit tests with
gradlew connectedObaGoogleDebugAndroidTest
to make sure you didn't break anythingIf you have multiple commits please combine them into one commit by squashing them for the initial submission of the pull request. When addressing comments on a pull request, please push a new commit per comment when possible (reviewers will squash and merge using GitHub merge tool)