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

Subscription block: it doesn't use the email I provide #39391

Open
oandregal opened this issue Sep 13, 2024 · 9 comments
Open

Subscription block: it doesn't use the email I provide #39391

oandregal opened this issue Sep 13, 2024 · 9 comments
Assignees
Labels
[Block] Subscriptions Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@oandregal
Copy link

Impacted plugin

Jetpack

Quick summary

The subscription block doesn't take into account the email you introduced if you are logged in to WordPress.com

Steps to reproduce

  1. Log in to your WordPress.com account with [email protected].
  2. Go to a site that uses the subscription block (or create a test site).
  3. Introduce an [email protected] in the subscription block and click "Subscribe".

The expectation is that YYY would be used as an email. Instead, what happened was that the WordPress.com account linked to the XXX was automatically subscribed.

A clear and concise description of what you expected to happen.

No response

What actually happened

No response

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

If Jetpack is going to use a different email for searching a WordPress.com account, it should be clear up-front. Perhaps the subscription block should display that logged-in WordPress.com account, or perhaps it should just use the email provided.

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@oandregal oandregal added [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged labels Sep 13, 2024
@github-actions github-actions bot added [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack [Pri] High labels Sep 13, 2024
@mrfoxtalbot
Copy link

I can replicate this. I entered [email protected]:

Screenshot 2024-09-13 at 13 50 08

...and the confirmation email showed [email protected]

Screenshot 2024-09-13 at 13 50 18

This is ONLY happening on AT sites. On Simple sites and self-hosted sites using Jetpack, the email shows grayed out and the field is not editable.

Screenshot 2024-09-13 at 14 00 26

@mrfoxtalbot mrfoxtalbot added Triaged [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Block] Subscriptions and removed Needs triage Ticket needs to be triaged labels Sep 13, 2024
@mrfoxtalbot
Copy link

It's very odd that it is only broken on AT sites.
I am keeping this as high-priority.

@oandregal
Copy link
Author

oandregal commented Sep 13, 2024

This is ONLY happening on AT sites. On Simple sites and self-hosted sites using Jetpack, the email shows grayed out and the field is not editable.

Oh, interesting, I didn't know this. I may have another piece of information to share then:

I was actually responding to a forum request. According to the HTML, that site is probably using a jetpack legacy widget that wraps a jetpack block subscription. You can visit https://b2b-marketing.pl/ and see how it doesn't provide the uneditable grayed-out email field even if you're logged in to WordPress.com.

Trying in my own site (jetpack connected, uses the jetpack subscription form), I do see the email uneditable.

Copy link
Contributor

github-actions bot commented Sep 13, 2024

Support References

This comment is automatically generated. Please do not edit it.

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Sep 13, 2024
@m1r0
Copy link
Member

m1r0 commented Sep 13, 2024

I did an investigation and landed on this diff: D128616-code

It looks like this logic is overwriting the initial email when jwt_token is passed but I'm missing context why that is.

I'm EOD, so feel free to pick this up.

cc @chrismccluskey

@jeherve
Copy link
Member

jeherve commented Sep 18, 2024

This was also reported in #39267:

5695723-zd-a8c
5829671-zd-a8c
5924135-zd-a8c
8697299-zd-a8c

@foleynotrose
Copy link

It's very odd that it is only broken on AT sites.

Are we certain? I observed the behavior on https://nonviolentleadership.com/, which has a Business subscription, but is not registering in our system as Atomic:

Pc9OEs-v-p2

Reported it in this other issue: #39267 (comment)

@heavyweight
Copy link
Contributor

I created this diff to fix it D161772-code
However I'm a bit hesitant for possible regressions.

Can someone try it out?

@fushar
Copy link
Contributor

fushar commented Sep 19, 2024

The above diff tests well for me. However, I'm not 100% sure if it won't cause other issues, so it would be better if more relevant team reviews it as well 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Subscriptions Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended
Projects
Development

No branches or pull requests

7 participants