Skip to content

[HACK Week] Attach application logs as attachment for support requests#16445

Merged
adborbas merged 5 commits into
trunkfrom
adborbas/woomob-1800
Dec 10, 2025
Merged

[HACK Week] Attach application logs as attachment for support requests#16445
adborbas merged 5 commits into
trunkfrom
adborbas/woomob-1800

Conversation

@adborbas

@adborbas adborbas commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Resolves WOOMOB-1800

Description

Attaches the full version of the application logs to the Zendesk issue when opening a support request. The trimmed application logs are still added as a custom field.

Test Steps

⚠️ This is an E2E test. You'll need to resolve the opened issue in Zendesk.

  1. Download the app from the build. You need the PR build as it contains the keys for creating the request.
  2. Login with any store.
  3. Navigate to Settings/Help & Support/Contact Support
  4. Fill the required values.
  5. Note that site address is prefilled.
  6. Tap "Submit Support Request"
  7. Open Zendesk and locate the issue.
  8. Validate that application logs are visible in the left side bar
  9. Validate that logs attached as well.

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@adborbas adborbas added this to the 23.9 milestone Dec 9, 2025
@adborbas
adborbas force-pushed the adborbas/woomob-1800 branch from 3e794b8 to 8ec0595 Compare December 9, 2025 12:45
@wpmobilebot

wpmobilebot commented Dec 9, 2025

Copy link
Copy Markdown
Collaborator

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr16445-7dd5a8a
Version23.8
Bundle IDcom.automattic.alpha.woocommerce
Commit7dd5a8a
Installation URL5fe0hfqhtk1i8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@adborbas adborbas changed the title Adborbas/woomob 1800 [HACK Week] Attach application logs as attachment for support requests Dec 9, 2025
@adborbas
adborbas marked this pull request as ready for review December 9, 2025 13:31
@itsmeichigo itsmeichigo self-assigned this Dec 10, 2025

@itsmeichigo itsmeichigo 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.

I tested and confirmed that the app log is attached in the request:

Image

Code looks great. Good job! 🚢

}
}

class DefaultApplicationLogProvider: ApplicationLogProvider {

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.

Nit: should we mark this class final if we don't intend to subclass it?

@itsmeichigo

Copy link
Copy Markdown
Contributor

I forgot - please update the release notes to ensure this issue is tested during beta testing too.

@adborbas

Copy link
Copy Markdown
Contributor Author

I forgot - please update the release notes to ensure this issue is tested during beta testing too.

I've explicitly decided not to add it there as the box says "warrants user-facing release notes" and since this change is not user-facing, I chose not to update the release notes.

I'm open to updating the release notes, but then we should also improve the box's text to reflect our intentions better.

@dangermattic

Copy link
Copy Markdown
Collaborator
1 Warning
⚠️ This PR is assigned to the milestone 23.9. This milestone is due in less than 2 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

@adborbas
adborbas merged commit a30d84d into trunk Dec 10, 2025
13 checks passed
@adborbas
adborbas deleted the adborbas/woomob-1800 branch December 10, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants