woocommerce-ios
woocommerce-ios copied to clipboard
[Engage] Universal Links: Add the woocommerce.com app links associated domain entitlement
Part of: #7535
Description
With this PR we add the associated domains entitlement to the Woo iOS app. Since we didn't upload the apple-app-site-association file to woocommerce.com, we cannot test this yet. Once we have this done, we can test the already implemented universal links, order details, and payments.
Testing instructions
Order Details
- Send yourself an email containing a link with the format https://woocommerce.com/orders/details?blog_id=your storeID&order_id=an existing orderID
- Tap on it. You can also test the switch to a different store by adding a store id to the link other than the currently selected in Woo app.
Open link with an order belonging to a store already selected in the app
https://user-images.githubusercontent.com/1864060/187661029-73b218a0-f28f-44cb-9da8-5fae53eb31cf.mov
Open link with an order belonging to a store not selected in the app. Switch store
https://user-images.githubusercontent.com/1864060/187661149-cf24c898-a9d1-4201-bde4-f3ae3f7384a7.mov
Payments
- Send yourself an email containing a link with the format https://woocommerce.com/payments
- Tap on it. Make sure it opens the Payments Hub Menu in the Woo Mobile app
https://user-images.githubusercontent.com/1864060/187913313-b126a79e-0b71-4f39-86ec-8a49acbe6f02.MP4
Screenshots
- [ ] I have considered if this change warrants user-facing release notes and have added them to
RELEASE-NOTES.txt
if necessary.
You can test the changes from this Pull Request by:
- Clicking here or scanning the QR code below to access App Center
- Then installing the build number
pr7632-a5f0456
on your iPhone
hi @Ecarrion Thanks for checking that :) I updated the PR description with the new format for the links, as we agreed to add the /mobile/
segment so redirections can happen easily. So, for instance, instead of https://woocommerce.com/payments
, it is https://woocommerce.com/mobile/payments
. Please try again with that links URLs formats and sorry for the confusion.
@toupper I'm still not getting re-directed. 🤔 is there a way for me to check why the app is not being invoked?
Thanks for trying again @Ecarrion :)
There are two points you can do to make sure you have the right configuration in your phone:
- Be sure you run the app from that branch (just in case ;)
- It might be that you tapped the small button on the device status bar while using the app, which led you back to woocommerce.com. Once you do that, the system would remember that and would always launch Safari when a universal link to your app is tapped. To undo that, long-tap the link, and in the menu that pops up, choose "Open with Woo"
If none of these work, we can schedule a call to check it further :)
Warnings | |
---|---|
:warning: | This PR is assigned to a milestone which is closing in less than 2 days Please, make sure to get it merged by then or assign it to a later expiring milestone |
Generated by :no_entry_sign: dangerJS
Thanks for checking that again @Ecarrion :)
The problem was on my end, when I updated the links to the add mobile path, the string visible to me changed, but the underlying hyperlink remained the same. 🤦
Yeah, that happened to me as well 😅