Skip to content

Show logs button when Studio ask to contact support - #765

Merged
wojtekn merged 2 commits into
trunkfrom
update/log-dialogs-to-show-logs-button
Jan 3, 2025
Merged

Show logs button when Studio ask to contact support#765
wojtekn merged 2 commits into
trunkfrom
update/log-dialogs-to-show-logs-button

Conversation

@wojtekn

@wojtekn wojtekn commented Jan 2, 2025

Copy link
Copy Markdown
Contributor

Related issues

Proposed Changes

I propose to display a button that allows users to open Studio logs for error messages that ask to contact support.

Screenshot 2025-01-02 at 15 06 22

Testing Instructions

  1. Create a Studio site
  2. Connect the site to the WordPress.com site
  3. Open the site in the editor and add die('test') in the wp-config.php
  4. Push the site in Sync tab or Export entire site in the Import / Export tab
  5. Confirm that dialog displays clear error and allows to open log file automatically

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@wojtekn
wojtekn requested a review from a team January 2, 2025 14:08
@wojtekn wojtekn self-assigned this Jan 2, 2025

@ivan-ottinger ivan-ottinger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The proposed changes look good and work as expected. 👌🏼🙂

Markup on 2025-01-02 at 17:38:58

One thing we could consider is to clarify how the user can contact support. As an example, we could consider adding a separate Contact us button or include a support email address.

Related discussion: https://github.com/Automattic/dotcom-forge/issues/10143#issuecomment-2568068469.

@wojtekn

wojtekn commented Jan 3, 2025

Copy link
Copy Markdown
Contributor Author

One thing we could consider is to clarify how the user can contact support. As an example, we could consider adding a separate Contact us button or include a support email address.

There is a Help menu with a few options. Let's leave it as is for now, and we may add fully customized error dialog in the future if needed.

@wojtekn
wojtekn merged commit 37a5608 into trunk Jan 3, 2025
@wojtekn
wojtekn deleted the update/log-dialogs-to-show-logs-button branch January 3, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants