-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Dashboard frontend enhancements #21598
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
2bf29e3 to
4d207d3
Compare
| return [ | ||
| 'routes' => [ | ||
| ['name' => 'dashboard#index', 'url' => '/', 'verb' => 'GET'], | ||
| ['name' => 'dashboard#updateLayout', 'url' => '/layout', 'verb' => 'POST'], |
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.
to be more compliant to what REST is about I would suggest to restructure this to be a resource, like simply /dashboard/layout and POST the data there
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.
I'd look into this in a separate pr since my first attempt for that didn't work out as expected.
ea46d1a to
2ae0c7a
Compare
4d207d3 to
adab679
Compare
1026be8 to
6b2bb32
Compare
|
@juliushaertl needs a rebase. :) Also:
I would say we make this the default app in master so we can properly dogfood it. We can still decide later before the RCs or so to keep Files the default. Ok @karlitschek @rullzer? |
|
@jancborchardt agreed |
9514283 to
7ca8c31
Compare
dbef1e2 to
10295d8
Compare
60041fb to
bdf7257
Compare
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Jan C. Borchardt <[email protected]>
Signed-off-by: Jan C. Borchardt <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Jan C. Borchardt <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
e63bb35 to
258cde1
Compare
Signed-off-by: Morris Jobke <[email protected]>
|
I reverted the URL change in the acceptance tests. The dashboard app should be disabled in those anyways. So this should fix those failing tests. |
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 31436: failurejsunitShow full logacceptance-app-files
Show full log |
|
Tests failures unrelated |
Follow up PR to #21346