Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

dashboard view #6

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

dashboard view #6

wants to merge 14 commits into from

Conversation

arnaudmm
Copy link
Contributor

Proposal about a dashboard view and a list view (filter and sort enable)

@sagikazarmark
Copy link
Contributor

Awesome work! It is not ugly at all.

I see now better how you imagined a complex layout.

As I said, currently I am struggling with ajax/not ajax solution, so I am not quite sure. I think the ideal would be to allow client side sorting, so that the data can be sorted/filtered without further requests to the server. However that would require an exposed REST API which might not be a very good idea.

Let me think about it a few days before deciding the fate of this PR. However the "global" layout you call will definitely be part of the core monitor, but before merging I have to decide whether we are going to use sync or async solution.

@arnaudmm
Copy link
Contributor Author

Got it!

2015-05-12 16:09 GMT+02:00 Márk Sági-Kazár notifications@github.com:

Awesome work! It is not ugly at all.

I see now better how you imagined a complex layout.

As I said, currently I am struggling with ajax/not ajax solution, so I am
not quite sure. I think the ideal would be to allow client side sorting, so
that the data can be sorted/filtered without further requests to the
server. However that would require an exposed REST API which might not be a
very good idea.

Let me think about it a few days before deciding the fate of this PR.
However the "global" layout you call will definitely be part of the core
monitor, but before merging I have to decide whether we are going to use
sync or async solution.


Reply to this email directly or view it on GitHub
#6 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants