-
Notifications
You must be signed in to change notification settings - Fork 478
[feat] Fix broken folders and Add apps dropdown in side menu #3685
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
[feat] Fix broken folders and Add apps dropdown in side menu #3685
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds an “apps” dropdown to the sidebar and enhances folder/app navigation by wiring folder assignment through the legacy API and improving prompts breadcrumb navigation.
Changes:
- Add a sidebar “apps” dropdown and split sidebar items into project vs app sections.
- Add sibling-folder dropdown navigation to the Prompts breadcrumb.
- Propagate
folder_idthrough legacy app outputs and update paths to support folder-based organization.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| web/oss/src/components/pages/prompts/components/PromptsBreadcrumb.tsx | Adds sibling-folder dropdowns to breadcrumb segments and updates action menu styling. |
| web/oss/src/components/Sidebar/types.d.ts | Replaces header with isAppSection to support sidebar sectioning. |
| web/oss/src/components/Sidebar/hooks/useSidebarConfig/index.tsx | Marks app-specific links with isAppSection and removes the prior “header” entry. |
| web/oss/src/components/Sidebar/components/SidebarMenu.tsx | Removes rendering logic for the old header group item. |
| web/oss/src/components/Sidebar/components/ListOfApps.tsx | New dropdown component that lists apps grouped by folders. |
| web/oss/src/components/Sidebar/Sidebar.tsx | Renders project menu, app dropdown + app menu section, then bottom menu. |
| api/oss/src/services/legacy_adapter.py | Ensures app updates preserve existing fields and includes folder_id in legacy outputs. |
| api/oss/src/dbs/postgres/git/dao.py | Attempts to persist folder_id on artifact edits. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
web/oss/src/components/pages/prompts/components/PromptsBreadcrumb.tsx
Outdated
Show resolved
Hide resolved
web/oss/src/components/pages/prompts/components/PromptsBreadcrumb.tsx
Outdated
Show resolved
Hide resolved
web/oss/src/components/pages/prompts/components/PromptsBreadcrumb.tsx
Outdated
Show resolved
Hide resolved
…umb.tsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…umb.tsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…umb.tsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mmabrouk, here is proof :D
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
web/oss/src/components/pages/prompts/components/PromptsBreadcrumb.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


Uh oh!
There was an error while loading. Please reload this page.