Skip to content

fix(lightspeed): show mcp settings only visible in chat tab - #2998

Merged
its-mitesh-kumar merged 2 commits into
redhat-developer:mainfrom
ciiay:rhdhbugs-3043-fix-mcp-settings-display-logic
May 5, 2026
Merged

fix(lightspeed): show mcp settings only visible in chat tab#2998
its-mitesh-kumar merged 2 commits into
redhat-developer:mainfrom
ciiay:rhdhbugs-3043-fix-mcp-settings-display-logic

Conversation

@ciiay

@ciiay ciiay commented May 5, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

For RHDHBUGS-3043, RHDHBUGS-3054 and RHDHBUGS-3071

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

screen recording (updated on 5/5)

rhdhbugs_3043.mp4

Test steps:
Please follow steps on PR description of #2754

@rhdh-gh-app

rhdh-gh-app Bot commented May 5, 2026

Copy link
Copy Markdown

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed workspaces/lightspeed/plugins/lightspeed patch v2.6.2

@ciiay
ciiay force-pushed the rhdhbugs-3043-fix-mcp-settings-display-logic branch from cb12a6c to efa63d9 Compare May 5, 2026 05:07
@codecov

codecov Bot commented May 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.21053% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.78%. Comparing base (2025174) to head (b12f312).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2998      +/-   ##
==========================================
+ Coverage   60.74%   60.78%   +0.03%     
==========================================
  Files        2067     2067              
  Lines       63800    63873      +73     
  Branches    16635    16664      +29     
==========================================
+ Hits        38758    38826      +68     
- Misses      24769    24774       +5     
  Partials      273      273              
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 2025174
ai-integrations 70.03% <ø> (ø) Carriedforward from 2025174
app-defaults 69.60% <ø> (ø) Carriedforward from 2025174
augment 69.36% <ø> (ø) Carriedforward from 2025174
bulk-import 72.44% <ø> (ø) Carriedforward from 2025174
cost-management 16.49% <ø> (ø) Carriedforward from 2025174
dcm 32.85% <ø> (ø) Carriedforward from 2025174
extensions 61.42% <ø> (ø) Carriedforward from 2025174
global-floating-action-button 73.75% <ø> (ø) Carriedforward from 2025174
global-header 61.62% <ø> (ø) Carriedforward from 2025174
homepage 50.84% <ø> (ø) Carriedforward from 2025174
konflux 91.01% <ø> (ø) Carriedforward from 2025174
lightspeed 69.96% <84.21%> (+0.31%) ⬆️
mcp-integrations 81.59% <ø> (ø) Carriedforward from 2025174
orchestrator 33.15% <ø> (ø) Carriedforward from 2025174
quickstart 62.64% <ø> (ø) Carriedforward from 2025174
sandbox 79.49% <ø> (ø) Carriedforward from 2025174
scorecard 83.61% <ø> (ø) Carriedforward from 2025174
theme 64.54% <ø> (ø) Carriedforward from 2025174
translations 8.49% <ø> (ø) Carriedforward from 2025174
x2a 82.09% <ø> (ø) Carriedforward from 2025174

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2025174...b12f312. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ciiay ciiay self-assigned this May 5, 2026
@its-mitesh-kumar

Copy link
Copy Markdown
Member

@ciiay notebook tab should come only in fullscreen, in 1.10 we are not going to have notebook in overlay or dock mode .

@ciiay
ciiay requested a review from a team as a code owner May 5, 2026 16:13
ciiay added 2 commits May 5, 2026 12:13
Signed-off-by: Yi Cai <yicai@redhat.com>
Signed-off-by: Yi Cai <yicai@redhat.com>
@ciiay
ciiay force-pushed the rhdhbugs-3043-fix-mcp-settings-display-logic branch from b1fdee2 to b12f312 Compare May 5, 2026 16:13
@sonarqubecloud

sonarqubecloud Bot commented May 5, 2026

Copy link
Copy Markdown

@its-mitesh-kumar its-mitesh-kumar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Working as per expected.
/lgtm

Screen.Recording.2026-05-05.at.10.15.05.PM.mov

@openshift-ci openshift-ci Bot added the lgtm label May 5, 2026
@its-mitesh-kumar its-mitesh-kumar changed the title fix(lightspeed): keep Chat/Notebooks tab across display modes fix(lightspeed): show mcp settings only visible in chat tab May 5, 2026
@its-mitesh-kumar
its-mitesh-kumar merged commit d4d74cf into redhat-developer:main May 5, 2026
60 checks passed
lokanandaprabhu pushed a commit to lokanandaprabhu/rhdh-plugins that referenced this pull request May 14, 2026
…eveloper#2998)

* fix(lightspeed): keep Chat/Notebooks tab across display modes

Signed-off-by: Yi Cai <yicai@redhat.com>

* addressed review comments

Signed-off-by: Yi Cai <yicai@redhat.com>

---------

Signed-off-by: Yi Cai <yicai@redhat.com>
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.

2 participants