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

Place incorporation date and state fields in the Company information form on separate rows to avoid overflowing labels #5923

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

MariaHCD
Copy link
Contributor

Details

Fix the overflowing labels for the Spanish translations of the Incorporation date and Incorporation state by having each field on a separate row instead of on the same row.

Screen Shot 2021-10-18 at 12 21 41 PM

Fixed Issues

(Partially fixes) #5904

Tests/ QA Steps

  1. Change language preference to Spanish by going to profile > preferences > language
  2. Navigate to the Company information step of the bank account flow by going to /bank-account/company or Workspace > Connect bank account
  3. Verify that the labels for the date and state picker are clearly visible

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Screen Shot 2021-10-18 at 12 17 08 PM

Mobile Web

Screen Shot 2021-10-18 at 12 36 35 PM

Desktop

Screen Shot 2021-10-18 at 12 40 42 PM

iOS

Android

@MariaHCD MariaHCD requested review from Gonals and a team October 18, 2021 08:42
@MariaHCD MariaHCD self-assigned this Oct 18, 2021
@MelvinBot MelvinBot requested review from Dal-Papa and removed request for a team October 18, 2021 08:42
@MariaHCD
Copy link
Contributor Author

Bump @Gonals :)

@Gonals Gonals merged commit c7812ac into main Oct 19, 2021
@Gonals Gonals deleted the maria-fix-incorporation-state-and-date-fields branch October 19, 2021 14:18
@OSBotify
Copy link
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.

@roryabraham
Copy link
Contributor

Hi @MariaHCD and @Gonals, thanks for fixing this! Since it fixes a deploy blocker, we probably should've CP'd it to staging by applying the CP Staging label before merging. So I'm going to go ahead and manually CP this now 🙂

github-actions bot pushed a commit that referenced this pull request Oct 19, 2021
…-and-date-fields

Place incorporation date and state fields in the Company information form on separate rows to avoid overflowing labels

(cherry picked from commit c7812ac)
@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by @roryabraham in version: 1.1.8-5 🚀

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

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@isagoico
Copy link

@roryabraham We retested this on Web, Android, mWeb and Desktop and it was a pass! Looks like iOS failed and that's why we haven't received the 1.1.8-5 build.

@roryabraham
Copy link
Contributor

Verified on iOS 1.1.8.8

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.8-9 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @Gonals in version: 1.1.8-10 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀

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.

None yet

6 participants