-
Notifications
You must be signed in to change notification settings - Fork 3k
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
#4375 fix ios user redirect to set up password + update staging url for app association on native platforms #4418
#4375 fix ios user redirect to set up password + update staging url for app association on native platforms #4418
Conversation
ios/ExpensifyCash/Chat.entitlements
Outdated
@@ -6,9 +6,8 @@ | |||
<string>development</string> | |||
<key>com.apple.developer.associated-domains</key> | |||
<array> | |||
<string>applinks:www.expensify.cash</string> | |||
<string>applinks:staging.expensify.cash</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<string>applinks:staging.expensify.cash</string> | |
<string>applinks:staging.new.expensify.com</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make this change on android too? Thanks: https://github.com/Expensify/App/blob/main/android/app/src/main/AndroidManifest.xml#L58
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, the requested changes have been implemented.
Code looks great, can you add a note on how to test the latest commit e203903? Thanks. |
Hello, I don't think you can test it. The magic link in emails is always for new.expensify.com, don't think you can generate one specifically for staging. Or do you mean to test app association with any link, not magic link for password set/reset specifically? |
You can ask the tester to copy the |
OK, I will add that later. Not on my laptop right now. |
@AndrewGable Hello, I have updated the original post with additional videos and instructions. Also the title is updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you!
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @AndrewGable in version: 1.0.83-2 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
Details
This PR updates the list of associated domains with iOS app to make it identical with the one in AndroidManifest.xml
Also updates the staging url used for app association on both iOS and android.
Fixed Issues
$ #4375
Tests / QA Steps
prod url test for iOS only:
staging url test (ios and android):
staging
to the URL, so it saysstaging.new.expenisify.com...
.Tested On
Screenshots
iOS
prod url:
4375-ios.mp4
staging url:
4375-ios-staging.mp4
Android
4357-andoid-staging.mp4