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

Bug: Tapping the Like link from email does not work #22212

Open
dvdchr opened this issue Dec 12, 2023 · 2 comments
Open

Bug: Tapping the Like link from email does not work #22212

dvdchr opened this issue Dec 12, 2023 · 2 comments

Comments

@dvdchr
Copy link
Contributor

dvdchr commented Dec 12, 2023

Refs p1702389743118429-slack-C0180B5PRJ4

Expected behavior

After tapping the Like link from a WP.com Post email notification, the post should be Liked.

Actual behavior

The post opens in Jetpack, but not Liked.

Steps to reproduce the behavior

  • Have the Jetpack app installed, and logged in with the same account on the browser.
  • From your email client, open an email notification for a post.
  • Tap the Like link.
  • Observe that it first opens Safari and then redirects to the Jetpack app, opening the post.
  • Observe that the post is not Liked.
Tested on any device, any supported version, Jetpack iOS 23.8
@dvdchr
Copy link
Contributor Author

dvdchr commented Dec 12, 2023

Investigation notes

  • The 'Like' link from email has the format of https://public-api.wordpress.com/bar with a lot of parameters.
  • The Jetpack app does not handle /bar routes, so it is expected for the user to visit Safari first.
  • The redirect_to parameter contains the value of https://<site-name>.wordpress.com/:yyyy/:mm/:dd/:slug, which is essentially a Reader Post link with some additional parameters embedded to it. Notable parameters in the link are like_actor, like_hmac, postid, like, and user.
  • We currently do not handle these parameters, although we do keep the parameters accessible throughout the session. For reference, the entry point for this URL is from this ReaderRoute handler, and the URL is passed off to the ReaderDetailCoordinator.

@dangermattic
Copy link
Collaborator

dangermattic commented Jan 4, 2025

Thanks for reporting! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants