Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.
https://jetpack.com/
Other
1.59k stars 800 forks source link

Feedback: Export to Google Sheets fails silently #28093

Open brbrr opened 1 year ago

brbrr commented 1 year ago

Impacted plugin

Jetpack

Steps to Reproduce

  1. Make sure your site have some form submissions
  2. Connect Google Drive from Calypso's UI via Tools > Marketing > Connections
  3. From /wp-admin/edit.php?post_type=feedback click "Export"
  4. "Export" to Google Sheets

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

Modal should display an error about failed export. Ideally, suggesting the user a way to fix the issue.

What actually happened

Nothing visually happened, but after exploring JS console, I noticed that admin-ajax.php (with grunion_export_to_gdrive payload) call returned a failure: {"success":false,"data":{"errors":{"failed_to_fetch_data":["Unable to fetch the requested data."]},"error_data":{"failed_to_fetch_data":{"status":500}}}}

Apparently, the above error was caused by a lack of required permissions for Drive connection. I haven't checked the second box when I was setting up the connection.

Browser

Google Chrome/Chromium

Other information

No response

Platform (Simple, Atomic, or both?)

Atomic

Reproducibility

Consistent

Severity

None

Available workarounds?

None

Workaround details

No response

github-actions[bot] commented 1 year ago

This issue has been marked as stale. This happened because:

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.