-
Notifications
You must be signed in to change notification settings - Fork 899
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@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? |
that was never discussed in the design ticket, and would require more terraform provider and backend changes 😅 |
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
to
c200325
Compare
c200325
to
685848d
Compare
Closes #8237