Skip to content

feat: add compose pay button and processing - #1683

Merged
saralvasquez merged 2 commits into
card-fields-compose-featurefrom
compose-pay-support
Sep 4, 2026
Merged

feat: add compose pay button and processing#1683
saralvasquez merged 2 commits into
card-fields-compose-featurefrom
compose-pay-support

Conversation

@saralvasquez

Copy link
Copy Markdown
Contributor

Summary of changes

  • adds pay button
  • adds backend processing like card client init and submit on pay button click
  • resolves a UI bug for the hint float height after recomposition
Screen_recording_20260819_115504.webm

AI Usage

Which AI Agent Was Used?

  • Copilot
  • Claude
  • Other (Type Name Here)

How was AI used?
Used the find cause of UI bug, wire back end processing, and write unit tests

Estimated AI Code Contribution

  • less than 30%
  • 30 - 60%
  • 60 - 100%

Checklist

  • Added a changelog entry
  • Tested and confirmed payment flows affected by this change are functioning as expected

Authors

@saralvasquez
saralvasquez requested a review from a team as a code owner August 19, 2026 18:57
@noguier
noguier force-pushed the compose-pay-support branch from 64883be to 54ab503 Compare August 25, 2026 14:25
@saralvasquez
saralvasquez force-pushed the compose-pay-support branch 2 times, most recently from 67a4fd2 to 6044262 Compare August 25, 2026 17:28
Comment thread Demo/src/main/java/com/braintreepayments/demo/ComposeCardFieldsFragment.kt Outdated
Base automatically changed from compose-cvv to card-fields-compose-feature August 26, 2026 16:29

@noguier noguier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a couple of non-blocking comments. Everything looks and functions great :)

@saralvasquez
saralvasquez force-pushed the compose-pay-support branch 2 times, most recently from 24cae10 to 9fa9bd5 Compare August 26, 2026 21:28
* [setPaymentRequest]. The result is delivered to [callback].
*
* If called before [initialize], delivers a [CardFieldsResult.Failure].
*/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These comments are very helpful!

@saralvasquez

Copy link
Copy Markdown
Contributor Author

Okie dokie! Take a look at the latest commit. I amended it so there's a lot in there but I made some changes to how rememberCardFieldsController works. It mostly functions as a constructor this way. I was going to rename it to CardFieldsController but apparently the remember keyword is a compose standard naming convention to signal that it's remembered on the back end so the robot suggested I don't change it

@saralvasquez
saralvasquez merged commit bce4780 into card-fields-compose-feature Sep 4, 2026
13 of 30 checks passed
@saralvasquez
saralvasquez deleted the compose-pay-support branch September 4, 2026 22:56
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.

4 participants