Conversation
🎩 To tophat this PR:You can add the following URL parameter to your browser to tophat this PR: |
e60bdef to
2bd5ec1
Compare
261ac09 to
4409f2f
Compare
2bd5ec1 to
0f426c7
Compare
4409f2f to
347c858
Compare
0f426c7 to
0e979c2
Compare
75b78e3 to
f02909a
Compare
e8fe5c7 to
4134ea7
Compare
f02909a to
c4f3cc7
Compare
4134ea7 to
7c60f5f
Compare
2332889 to
e650043
Compare
7c60f5f to
a81f654
Compare
e650043 to
c965cc7
Compare
a81f654 to
c1b63c6
Compare
c965cc7 to
b71f997
Compare
c1b63c6 to
2892744
Compare
Merge activity
|
|
Given the discussion we've had we should look to collect all deferred feedback so we don't lose it while we do final touch-ups in the next week |
2892744 to
84b41ed
Compare
b71f997 to
6e41b96
Compare
84b41ed to
5de008b
Compare
6e41b96 to
0bdf1e1
Compare
|
> Can we put as much as possible under "/app"? |
0bdf1e1 to
d437176
Compare
| export const QUICK_START_PATH = "/quick-start"; | ||
| const SETTINGS_PATH = "/settings"; | ||
| const IMPORT_PATH = "/app/editor/import-pipeline"; | ||
| const DASHBOARD_PATH = "/dashboard"; |
There was a problem hiding this comment.
>/pipelines - My Pipelines
>/components - Components
>/favorites - Favorites
>/recently-viewed - Recently Viewed
Is it possible to limit the number of root route prefixes? having so many prefixes makes it very hard to route backend and frontend. Especially in local and HuggingFace deployments.
Can we put as much as possible under "/app"?

Description
This pull request enables the Dashboard feature by default and restructures the routing to make Dashboard the primary navigation experience. The Dashboard flag is now enabled by default, and all Dashboard routes have been moved to top-level paths (removing the
/dashboardprefix). The routing logic has been updated to use a pathless layout pattern where the Dashboard layout wraps all routes when the flag is enabled, and falls back to the original Home component when disabled.Related Issue and Pull requests
Type of Change
Checklist
Screenshots (if applicable)
Test Instructions
//- Dashboard home/pipelines- My Pipelines/runs- All Runs/components- Components/favorites- Favorites/recently-viewed- Recently ViewedAdditional Comments
This is a breaking change as it modifies the URL structure for Dashboard routes. Users with bookmarks to
/dashboard/*paths will need to update them to the new top-level paths.