-
Notifications
You must be signed in to change notification settings - Fork 325
Dan product left nav update #1445
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
…ing, add dashbaord styling, fmt, lint
pub mod snapshots; | ||
pub mod uploader; | ||
|
||
#[get("/deployments")] |
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.
This conflicts with our cloud route.
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.
This is not new code, I just moved all the deployment/engine stuff out of lib.rs and into an organized file structure. We add /dashboard to the front. So the actual url is /dashboard/deployments, soon to be /engine/deployments.
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.
Right, sorry. Been a while since I looked at this code.
This reverts commit 5ce41e7.
Feedback will be added in another PR
Screen.Recording.2024-05-07.at.12.51.38.PM.1.mov