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

Fix maxLength data type #7086

Merged
merged 1 commit into from
Jan 7, 2022
Merged

Fix maxLength data type #7086

merged 1 commit into from
Jan 7, 2022

Conversation

luacmartins
Copy link
Contributor

@luacmartins luacmartins commented Jan 7, 2022

Details

Introduced here #6845 cc @marcochavezf

Fixes data type for maxLength prop.

Fixed Issues

$ #7085

Tests

  1. On Android, navigate to Settings > Payments > Add payment method > Debit card
  2. Verify there is no error thrown.
  3. Verify that the cvv and expiration date fields do not allow more than 4 and 7 digits, respectively.

QA Steps

Steps above.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

android-2.mov

@luacmartins luacmartins requested a review from a team as a code owner January 7, 2022 19:22
@luacmartins luacmartins self-assigned this Jan 7, 2022
@MelvinBot MelvinBot requested review from tylerkaraszewski and removed request for a team January 7, 2022 19:23
@luacmartins luacmartins changed the title fix maxLength type Fix maxLength type Jan 7, 2022
@luacmartins luacmartins changed the title Fix maxLength type Fix maxLength data type Jan 7, 2022
@marcochavezf
Copy link
Contributor

Woo, good catch! I didn't imagine that change could create a cast issue in RN. Thanks for addressing that Carlos! (I owe you 2 tacos hehe)

@luacmartins
Copy link
Contributor Author

Self-merging with approval!

@luacmartins luacmartins merged commit 3cd9f1b into main Jan 7, 2022
@luacmartins luacmartins deleted the cmartins-fixMaxLength branch January 7, 2022 20:14
@OSBotify
Copy link
Contributor

OSBotify commented Jan 7, 2022

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @luacmartins in version: 1.1.26-2 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @luacmartins in version: 1.1.27-0 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.27-1 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

3 participants