Skip to content

fix(site): hide "Show parent apps" when no running or starting devcontainers #19200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 6, 2025

Conversation

DanielleMaywood
Copy link
Contributor

@DanielleMaywood DanielleMaywood commented Aug 6, 2025

Fixes #19199

We now hide the "Show parent apps" button when there are no running or starting devcontainers.

@DanielleMaywood DanielleMaywood marked this pull request as ready for review August 6, 2025 11:02
Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍🏻. Would be nice to see a test case that verifies/enforces the behavior though.

@DanielleMaywood
Copy link
Contributor Author

I've added a storybook but I've noticed it doesn't actually show the devcontainer, and after running storybook in my workspace the existing Devcontainer story doesn't show any either. I think the change that introduced useAgentContainers may have broken the prior story.

I've tried fixing this myself, and with the aid of Claude Opus 4.1, but I've not had any luck. I believe the main issue is our component is handling both data fetching and rendering, whereas with other parts of the codebase we follow the pattern: FooBar paired with a FooBarView.

@aslilac Do you think we should follow the same approach here?

@aslilac
Copy link
Member

aslilac commented Aug 6, 2025

that's one approach, but you can also just mock the query response by setting the queries parameter on the story. for example:

@DanielleMaywood DanielleMaywood merged commit 1c70d32 into main Aug 6, 2025
30 of 32 checks passed
@DanielleMaywood DanielleMaywood deleted the danielle/devcontainer/show-parent-apps branch August 6, 2025 17:24
@github-actions github-actions bot locked and limited conversation to collaborators Aug 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: "Show parent apps" button visible but does not do anything
3 participants