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

Unable to Update Newsletter Settings #89109

Closed
upwardmomentum84 opened this issue Mar 30, 2024 · 10 comments
Closed

Unable to Update Newsletter Settings #89109

upwardmomentum84 opened this issue Mar 30, 2024 · 10 comments
Assignees
Labels
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. [Platform] Atomic [Plugin] Jetpack [Pri] High [Status] Priority Review Triggered Quality squad has been notified of this issue in #dotcom-triage-alerts Triaged To be used when issues have been triaged. [Type] Bug

Comments

@upwardmomentum84
Copy link

upwardmomentum84 commented Mar 30, 2024

Impacted plugin

Jetpack

Quick summary

On Atomic sites, the Newsletter settings are not saving properly. It shows the error "An unexpected error occurred. Please try again later" when saving. Refreshing the page, shows the settings are changed however, the live site remains unaffected.

Steps to reproduce

  1. On an Atomic site, navigate to Settings > Newsletter
  2. Adjust the settings under Subscriptions.
  3. Click "Save Settings". See the error.

error

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

That the settings would save successfully and the changes be reflected on the live site.

What actually happened

An error shows and the live site doesn't reflect the changes.

This is what a sample post page looks like with all Newsletter Subscriptions settings turned off.

Subscribe block showing

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

Platform (Simple and/or Atomic)

Atomic

Logs or notes

Reported here: 7966380-zd-a8c

Was not able to replicate on a Simple site.

@upwardmomentum84 upwardmomentum84 added [Type] Bug [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Plugin] Jetpack Needs triage Ticket needs to be triaged labels Mar 30, 2024
@github-actions github-actions bot added [Platform] Atomic [Pri] High [Status] Priority Review Triggered Quality squad has been notified of this issue in #dotcom-triage-alerts labels Mar 30, 2024
Copy link

github-actions bot commented Mar 30, 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 Mar 30, 2024
@jeherve
Copy link
Member

jeherve commented Apr 2, 2024

Moving this to Calypso since this is about the toggle in the Calypso UI.

@jeherve jeherve transferred this issue from Automattic/jetpack Apr 2, 2024
@jeherve
Copy link
Member

jeherve commented Apr 2, 2024

I can't quite reproduce on my end, the settings appear to save properly on my end. @upwardmomentum84 Could you tell us more about the site setup? Is the site public or private?

Could you also open the browser's dev tools, go to the "Network" tab, filter for "XHR", try reproducing the error, and then click on the call that generated the error and navigate to "Response"? What do you see there?

Thank you!

cc @pkuliga who recently worked on #88702.

@jeherve jeherve added [Status] Needs Author Reply Triaged To be used when issues have been triaged. and removed Needs triage Ticket needs to be triaged labels Apr 2, 2024
@pkuliga
Copy link
Contributor

pkuliga commented Apr 2, 2024

I was able to repro this on Atomic.

The GET https://public-api.wordpress.com/rest/v1.4/sites/229656926/settings?http_envelope=1 request does not contain the jetpack_subscriptions_subscribe_post_end_enabled setting which is causing this error.

It was added here Automattic/jetpack#36381 to the endpoint. Is it not on Atomic yet?

@pkuliga
Copy link
Contributor

pkuliga commented Apr 2, 2024

Ok, now it's there and it works ok. A cache maybe?

@upwardmomentum84
Copy link
Author

The site in the screenshots is an Atomic test site on the Creator plan. It only has minimal plugins active (Akismet, Jetpack, Gutenberg and WPCOM Editing Toolkit). The newsletter settings seem to be saving now and being applied on the live site (they weren't when I filed the bug report). I'm still seeing the error in Calypso when saving the changes though. The toggles also don't update until you refresh the page.

Here is a screen recording showing the error and some information from the browser dev tools:
https://github.com/Automattic/wp-calypso/assets/93800099/0d75361b-365f-4b81-bd5e-1eac2f9cead4

Here is the XHR Response for https://public-api.wordpress.com/rest/v1.2/sites/199097935?http_envelope=1
XHR-Response.txt

@pkuliga
Copy link
Contributor

pkuliga commented Apr 2, 2024

Thanks for it @upwardmomentum84!

Is it the case you have the public-api.wordpress.com sandboxed? If yes, I think that could be your sandbox cache not flushed, if not I need to talk to opers since others could also be affected.

@upwardmomentum84
Copy link
Author

No, it's not sandboxed. I don't even know how to set one of those up :)

@Aurorum
Copy link
Contributor

Aurorum commented Apr 13, 2024

I had a report of this in 8036390-zen in case it helps debug (not sure if it's the same cause though, and I can't submit an API request to Atomic sites!).

@enejb
Copy link
Member

enejb commented May 22, 2024

I am closing this issue since I am not able to replicate it any more. Feel free to open it again if this is not this issue still exist.

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. Previously known as "Happiness Request". [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Platform] Atomic [Plugin] Jetpack [Pri] High [Status] Priority Review Triggered Quality squad has been notified of this issue in #dotcom-triage-alerts Triaged To be used when issues have been triaged. [Type] Bug
Development

Successfully merging a pull request may close this issue.

5 participants