Skip to content

Conversation

@aleDsz
Copy link
Member

@aleDsz aleDsz commented Oct 31, 2025

No description provided.

@aleDsz aleDsz self-assigned this Oct 31, 2025
@aleDsz aleDsz changed the title Allow users to assign folders for their apps Allow users to assign folders to their apps Oct 31, 2025
Use folders to organize how how apps are displayed.
'''
}
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

What happens if there are no folders? We have two options:

  1. Don't show anything (so we don't need to check @teams_enabled)
  2. Show something saying they can create folders in the Livebook Teams interface (requires @teams_enabled checks)

cc @hugobarauna

Copy link
Member

Choose a reason for hiding this comment

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

Good question. I guess we can we can go with option 2:

Show something saying they can create folders in the Livebook Teams interface (requires @teams_enabled checks)

So that the folders feature gets discoverable from the Livebook side as well.

@josevalim what do you think?

Copy link
Member Author

Choose a reason for hiding this comment

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

What happens if there are no folders?

You can deploy without selecting an app folder, which will be in the Ungrouped apps "folder"

Copy link
Contributor

Choose a reason for hiding this comment

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

@hugobarauna i am fine either way. whatever you think is a better experience.

Copy link
Member

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

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

After Jose comments it looks good to me!

@aleDsz aleDsz merged commit 82b2b28 into main Nov 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants