Skip to content

fix(q): Remove Q Show Logs menu option from non Q views#8301

Merged
ashishrp-aws merged 2 commits intoaws:masterfrom
vpbhargav:master
Nov 18, 2025
Merged

fix(q): Remove Q Show Logs menu option from non Q views#8301
ashishrp-aws merged 2 commits intoaws:masterfrom
vpbhargav:master

Conversation

@vpbhargav
Copy link
Contributor

Description

Noticed that if Q extension is installed alongside AWS Toolkit, the Show Logs option redirecting to Q logs were showing up in the toolkit views as well. This does not make sense, especially for SMUS as it is misleading.

image

Change now shows this only for amazonq views similar to the learn more option.

Motivation

Remove misleading show logs option from non Q views

Testing Done

Tested locally that it no longer shows up in Toolkit but shows in Q.

image image
  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@vpbhargav vpbhargav requested a review from a team as a code owner November 17, 2025 17:43
@github-actions
Copy link

  • This pull request implements a feat or fix, so it must include a changelog entry (unless the fix is for an unreleased feature). Review the changelog guidelines.
    • Note: beta or "experiment" features that have active users should announce fixes in the changelog.
    • If this is not a feature or fix, use an appropriate type from the title guidelines. For example, telemetry-only changes should use the telemetry type.

**Description**

Noticed that if Q extension is installed alongside AWS Toolkit,
the Show Logs option redirecting to Q logs were showing up in
the toolkit views as well. This does not make sense, especially
for SMUS as it is misleading.

Change now shows this only for amazonq views similar to the learn more
option.

**Motivation**

Remove misleading show logs option from non Q views

**Testing Done**

Tested locally that it no longer shows up in Toolkit but
shows in Q.
@vpbhargav vpbhargav changed the title fix(q): Remove Q Show Logs options from non Q views fix(q): Remove Q Show Logs menu option from non Q views Nov 17, 2025
@ashishrp-aws ashishrp-aws merged commit 999b24a into aws:master Nov 18, 2025
45 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants