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

Subscriptions: Improve Messaging when an email has too many Pending subscriptions #16790

Closed
tmmbecker opened this issue Aug 10, 2020 · 3 comments
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it

Comments

@tmmbecker
Copy link

Users that have 50 pending subscriptions aren't able to subscribe to any more, but the error message is very vague:

Screen Shot on 2020-08-10 at 13-55-37

We fixed the error message for those trying to subscribe to a site for which they already have a pending sub in #14326, but the message users see when subscribing to a new site is still very confusing.

Steps to reproduce the issue

  1. Start with a WordPress.com account that has at least 50 pending subscriptions.
  2. Try to subscribe to a new site.
  3. See the confusing error.
  4. Remove some of those pending subscriptions from the account.
  5. Subscribe again successfully

What I expected

I'd expect to see a message indicating that I have too many pending subs and that's why I'm not able to subscribe, with a link to my subscriptions dashboard where I can remove those pending subs.

Internal ref: 3202604-zen

@tmmbecker tmmbecker added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" labels Aug 10, 2020
@htdat htdat changed the title Subscriptions: Improve Messaging when there are too many Pending subscriptions Subscriptions: Improve Messaging when an email has too many Pending subscriptions Aug 15, 2020
@htdat
Copy link
Member

htdat commented Aug 16, 2020

Actually, we have already handled this case with this PR #14275

There was an error when subscribing. Please try again.

This error message comes from the default case. Maybe another issue happens instead of so many pending subscriptions.

default : ?>
<p class="error"><?php esc_html_e( 'There was an error when subscribing. Please try again.', 'jetpack' ); ?></p>
<?php break;

@htdat
Copy link
Member

htdat commented Aug 16, 2020

Users that have 50 pending subscriptions.

Also, our limit is much less than this now. Ref for A12s 261bc-pb

@tmmbecker
Copy link
Author

I'll close this for now.

But it's a bit odd that I got that message, removed a bunch of the pending subs and then was able to subscribe with no error. If it were something else, I wouldn't have expected the error to clear after only removing the pending subs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. aka "Happiness Request", or "User Report" [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

No branches or pull requests

2 participants