- Notifications
You must be signed in to change notification settings - Fork906
feat: group apps together on workspace page#18018
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
@aslilac Left some comments, didn't approve yet because its still marked as draft. It would be useful to add a storybook test to be able to easily visualize the change. |
sorry about that, I realized it wasn't actually ready after I tagged you and marked it as ready 🤦♀️ it should actually be ready to go now |
Can we set up an icon for the group, too? |
aslilac commentedMay 29, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
that was never discussed in the design ticket, and would require more terraform provider and backend changes 😅 |
matifali commentedMay 29, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The next step would be to group all the IDEs we have in dogfood. It will be interesting to support this while our IDEs come from modules. |
I don't think we should group IDEs, I think we should use them to dogfood dynamic parameters. but we should have all of the modules take a |
e611845
toc200325
Comparec200325
to685848d
Compare232c72f
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Closes#8237