Skip to content
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

web-ui2 #12

Merged
merged 1 commit into from Aug 9, 2016
Merged

web-ui2 #12

merged 1 commit into from Aug 9, 2016

Conversation

Ivansss
Copy link
Member

@Ivansss Ivansss commented Aug 8, 2016

Updated web UI as discussed with @jancborchardt
Added active users statistics chart.
web-ui1
web-ui2
web-ui3

@schiessle
Copy link
Member

@Ivansss what's the state of this pull request? Are you still working on it? Or can we merge it so that I can transfer it to a admin settings page?

@Ivansss
Copy link
Member Author

Ivansss commented Aug 9, 2016

@schiessle you can merge it.

@jancborchardt
Copy link
Member

@schiessle @Ivansss if it’s included in the release, it should be on the admin settings page. Will this happen in time? :)

@schiessle
Copy link
Member

if it’s included in the release, it should be on the admin settings page. Will this happen in time? :)

If the new admin settings are ready for stable10 it will be in the release as a admin setting. If not it will be a stand-alone app... PR is already here and just waits: #18 😉

@jancborchardt
Copy link
Member

@schiessle either way it should be in the admin settings. It’s an admin functionality and should definitely not be in the apps menu. Especially since we know that we will move it around it will otherwise lead to confusion on update.

@schiessle
Copy link
Member

ihmo it's way to large to put it in the old admin settings, it is multiple pages on a normal screen

@schiessle
Copy link
Member

if we want it in the admin settings, we need to think about a stripped-down version. Having all the graphs in the middle of the admin settings doesn't work

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

Successfully merging this pull request may close these issues.

None yet

3 participants