woocommerce-ios icon indicating copy to clipboard operation
woocommerce-ios copied to clipboard

[Engage] Universal Links: Add the woocommerce.com app links associated domain entitlement

Open toupper opened this issue 1 year ago • 1 comments

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

  1. Send yourself an email containing a link with the format https://woocommerce.com/orders/details?blog_id=your storeID&order_id=an existing orderID
  2. 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

  1. Send yourself an email containing a link with the format https://woocommerce.com/payments
  2. 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.

toupper avatar Sep 01 '22 14:09 toupper

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
If you need access to App Center, please ask a maintainer to add you.

wpmobilebot avatar Sep 01 '22 14:09 wpmobilebot

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 avatar Oct 04 '22 08:10 toupper

@toupper I'm still not getting re-directed. 🤔 is there a way for me to check why the app is not being invoked?

Ecarrion avatar Oct 04 '22 22:10 Ecarrion

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 :)

toupper avatar Oct 05 '22 08:10 toupper

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

peril-woocommerce[bot] avatar Oct 05 '22 16:10 peril-woocommerce[bot]

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 😅

toupper avatar Oct 05 '22 16:10 toupper