Skip to content

Add a placeholder to the TaxID field#5535

Merged
NikkiWines merged 1 commit into
mainfrom
clem-taxid-placeholder
Sep 27, 2021
Merged

Add a placeholder to the TaxID field#5535
NikkiWines merged 1 commit into
mainfrom
clem-taxid-placeholder

Conversation

@Dal-Papa

Copy link
Copy Markdown

Details

Add a placeholder for the Tax ID field to encourage using 9 digits

Fixed Issues

Slack thread

Tests & QA

  1. Navigate to /bank-account/company
  2. Focus on the TaxID number field
  3. Verify that you now see a placeholder

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Screen Shot 2021-09-27 at 11 32 34

Mobile Web

Desktop

iOS

Android

@Dal-Papa Dal-Papa requested a review from a team September 27, 2021 09:33
@Dal-Papa Dal-Papa self-assigned this Sep 27, 2021
@MelvinBot MelvinBot requested review from NikkiWines and removed request for a team September 27, 2021 09:34

@NikkiWines NikkiWines 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.

:shipit:

@NikkiWines NikkiWines merged commit cf37f99 into main Sep 27, 2021
@NikkiWines NikkiWines deleted the clem-taxid-placeholder branch September 27, 2021 14:22
@OSBotify

Copy link
Copy Markdown
Contributor

✋ 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
Copy Markdown
Contributor

🚀 Deployed to staging by @NikkiWines in version: 1.1.2-9 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @timszot in version: 1.1.3-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