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

Encode secondary login URL #6479

Merged
merged 5 commits into from
Nov 25, 2021
Merged

Encode secondary login URL #6479

merged 5 commits into from
Nov 25, 2021

Conversation

luacmartins
Copy link
Contributor

Details

iOS treats URL encoding a bit differently and I had to adjust it to be able to deeplink into the secondary login page in OldDot.

Fixed Issues

$ #6475

Tests

  1. Launch the app and login with gmail account
  2. Initiate VBA flow and add an "Open" bank account. Instructions.
  3. At the last step tap "Add work email address"
  4. Verify that OldDot opens in the accounts page and the add secondary login modal opens.

QA Steps

  1. Launch the app and login with gmail account
  2. Initiate VBA flow and add an "Open" bank account. Instructions.
  3. At the last step tap "Add work email address"
  4. Verify that OldDot opens in the accounts page and the add secondary login modal opens.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

web.mov

Mobile Web

mweb.mov

Desktop

desktop.mov

iOS

ios.mov

Android

android.mov

@luacmartins luacmartins requested a review from a team as a code owner November 25, 2021 17:56
@luacmartins luacmartins self-assigned this Nov 25, 2021
@github-actions
Copy link
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

src/CONST.js Outdated Show resolved Hide resolved
@madmax330 madmax330 merged commit dd291dc into main Nov 25, 2021
@madmax330 madmax330 deleted the cmartins-encodeURL branch November 25, 2021 19:17
github-actions bot pushed a commit that referenced this pull request Nov 25, 2021
Encode secondary login URL

(cherry picked from commit dd291dc)
@isagoico
Copy link

@luacmartins hello! I think there were some conflicts while merging, we're not seeing build 1.1.16-9 #6481 (comment)

aldo-expensify added a commit that referenced this pull request Nov 25, 2021
@luacmartins
Copy link
Contributor Author

Thanks @isagoico! Conflicts resolved. Deploying here

@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by @madmax330 in version: 1.1.16-9 🚀

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

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

@isagoico
Copy link

isagoico commented Nov 25, 2021

App is crashing on iOS and Android when tapping on "Add work email..." button. Video below

crash.mp4

@isagoico
Copy link

isagoico commented Nov 26, 2021

Tested this on Android and iOS and it's a pass 🎉 No more crashes!

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.16-10 🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @madmax330 in version: 1.1.16-11 🚀

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

@ogumen
Copy link

ogumen commented Dec 2, 2021

The accessibility issues related to this PR:

  1. The "Add work email address" link is announced without role, just as plain text. The screen reader user is not informed that it opens a new browser tab - failure of WCAG SC 4.1.2
    VBA flow_Add work email address announced without role
  2. The grey Close button (#C6C9CA) on white background (#FFFFFF) has a color contrast ratio of 1.66:1, failing the minimum contrast requirements of 3.00:1 - failure of WCAG SC 1.4.11
  3. The grey Email or Phone placeholder text (#7D8B8F) on white background (#FFFFFF) has a color contrast ratio of 3.52:1, failing the minimum color contrast of 4.50:1 - failure of WCAG SC 1.4.3
    Secondary login_Grey button and placeholder text fail contrast requirements
  4. The white text (#FFFFFF) against green background (#03BB6D) has a color contrast ratio of 2.52:1, failing the minimum contrast requirements of 4.50:1 - failure of WCAG SC 1.4.3
    Secondary login_White text on green button fails contrast requirements
  5. The Close button and Privacy link have no visible outline on focus due to CSS outline:0 rule. The keyboard users will not be able to determine the currently focused element - failure of WCAG SC 2.4.7
    Secondary login_Close and Privacy elements have no visible outline on focus
  6. The light green focus outline (#80E8BC) against green button (#03BB6D) has a color contrast ratio of 1.70:1, failing the minimum contrast requirements of 3.00:1 - failure of WCAG SC 1.4.11
    Secondary login_Light green focus outline on green button fails contrast requirements
  7. The blue Privacy link (#0185FF) against white background (#FFFFFF) has a color contrast ratio of 3.62:1 failing the minimum color contrast requirements of 4.50:1 - failure of WCAG SC 1.4.3
    Secondary login_Blue link on white background fails contrast requirements
  8. The Close modal button is announced without any label, as "button" - failure of WCAG SC 4.1.2
    Secondary login_Close modal button announced without label
  9. Incorrect document role and no label provided for modal dialog - failure of WCAG SC 1.3.1
    Secondary login_Incorrect document role and no label for the dialog
  10. The Email or Phone text field uses placeholder text as the only label. Once user starts typing into the field, the placeholder text disappears and the field's purpose is no longer obvious - failure of WCAG SC 3.3.2
    Secondary login_The text field uses placeholder text as the only visible label
  11. The focus is not trapped in the modal dialog and the background elements can be focused with keyboard - failure of WCAG SC 2.4.3
Secondary.login_Focus.not.trapped.in.modal.dialog.mp4

@OSBotify
Copy link
Contributor

OSBotify commented Dec 6, 2021

🚀 Deployed to production by @roryabraham in version: 1.1.17-7 🚀

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.

6 participants