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 bad apple app site association #5077

Merged
merged 1 commit into from
Sep 7, 2021
Merged

Conversation

marcaaron
Copy link
Contributor

@marcaaron marcaaron commented Sep 3, 2021

Details

Fixes deep links and unblocks testing via deep links on iOS. Seems this has been broken for a minute. But we need QA to test this flow in the latest staging build so I'm asking for a CP.

Fixed Issues (Comment)

$ #5029 (comment)

Tests

Can't really test this on dev because the universal link needs to be tested on staging after staging web deploys. It uses the JSON file that we are modifying here.

QA Steps

  1. Navigate to https://staging.new.expensify.com/bank-account
  2. Verify the deep link works

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@marcaaron marcaaron self-assigned this Sep 3, 2021
@marcaaron marcaaron requested a review from a team as a code owner September 3, 2021 22:52
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2021

⚠️ ⚠️ 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.

@MelvinBot MelvinBot requested review from deetergp and removed request for a team September 3, 2021 22:53
@marcaaron
Copy link
Contributor Author

On second thought we found a workaround and don't need to CP this.

@marcaaron marcaaron changed the title [CP Staging] Fix bad apple app site association Fix bad apple app site association Sep 3, 2021
@marcaaron
Copy link
Contributor Author

Unsure if removing that label makes a CP go away... 😅

@deetergp deetergp merged commit e6ed8dd into main Sep 7, 2021
@deetergp deetergp deleted the marcaaron-fixiOSDeepLink branch September 7, 2021 18:17
@OSBotify
Copy link
Contributor

OSBotify commented Sep 7, 2021

✋ 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

OSBotify commented Sep 8, 2021

🚀 Deployed to staging by @deetergp in version: 1.0.94-2 🚀

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

@mvtglobally
Copy link

@marcaaron @deetergp Should we QA on all platforms?

@marcaaron
Copy link
Contributor Author

The only platforms we need to test this on are Android and iOS.

@marcaaron
Copy link
Contributor Author

The change really should only affect iOS

@OSBotify
Copy link
Contributor

OSBotify commented Sep 9, 2021

🚀 Deployed to production by @yuwenmemon in version: 1.0.95-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.

4 participants