Skip to content

fix(lsp): refresh installation status on the correct settings page#2482

Merged
bajrangCoder merged 2 commits into
mainfrom
fix/lsp-install-status
Jul 12, 2026
Merged

fix(lsp): refresh installation status on the correct settings page#2482
bajrangCoder merged 2 commits into
mainfrom
fix/lsp-install-status

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes LSP detail status refreshes on the settings page. The main changes are:

  • Exposes each settings page's own list element.
  • Tags LSP detail lists with their server id.
  • Refreshes the connected matching detail list after show and setting updates.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
src/components/settingsPage.js Adds a list-element accessor so callers can use the settings page's own list.
src/settings/lspServerDetail.js Refreshes LSP detail state against the page list for the matching server.

Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile

Comment thread src/settings/lspServerDetail.js Outdated
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder bajrangCoder merged commit 0bfb543 into main Jul 12, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Jul 12, 2026
@bajrangCoder bajrangCoder deleted the fix/lsp-install-status branch July 12, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant