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

PayTo - Component state #2025

Open
wants to merge 8 commits into
base: feature/payto
Choose a base branch
from

Conversation

araratthehero
Copy link
Contributor

Description

Generate component state for PayTo

Checklist

  • Code is unit tested
  • Changes are tested manually

COAND-1058

@araratthehero araratthehero requested a review from a team as a code owner February 26, 2025 13:29
@araratthehero araratthehero force-pushed the chore/payto_component_state branch from e3c3132 to 8a2ca0d Compare February 26, 2025 13:29
@AdyenAutomationBot AdyenAutomationBot added the Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes label Feb 26, 2025
@@ -21,11 +21,13 @@

<style name="AdyenCheckout.PayTo.FirstNameEditText" parent="AdyenCheckout.TextInputEditText">
<item name="android:hint">@string/checkout_payto_first_name_hint</item>
Copy link
Contributor

Choose a reason for hiding this comment

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

The most important input fields already have an autofill hint, but I think some others could also use it. The first and last name inputs for example. Here is a extended list of hints. You can either hardcode those values in xml or use this constants class through code

@araratthehero araratthehero force-pushed the chore/payto_component_state branch from 8a2ca0d to 2631b39 Compare February 27, 2025 08:38
Copy link
Contributor

✅ No public API changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore [PRs only] Indicates any task that does not need to be mentioned in the public release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants