ubersicht icon indicating copy to clipboard operation
ubersicht copied to clipboard

Adding weekly totals and fixing closed counters

Open daonb opened this issue 12 years ago • 11 comments

Great project, thanks! Our organization is a bit slow so the 24h counters were at 0 so I added weekly counters and hidden the 24h if they are at 0.

Benny

daonb avatar Mar 05 '14 23:03 daonb

@daonb you might want to set the org back to hoodiehq :)

svnlto avatar Mar 06 '14 08:03 svnlto

I thought you'd like that… That's the first thing the readme told me to do. Seriously, there has to be a better way to set the default organization.

daonb avatar Mar 06 '14 10:03 daonb

Seriously, there has to be a better way to set the default organization.

Yeah, there will be in the future. Mind, this whole thing was created in a hack session :)

svnlto avatar Mar 06 '14 10:03 svnlto

It's very impressive. If I change the organization I won't be able to use it for hasadna. Maybe we should get the organization from the url's domain if it's like xyz.github.io and hoodiehq if it's not under github.io?

daonb avatar Mar 06 '14 11:03 daonb

@daonb You're probably better off using your own fork - http://hasadna.github.io/ubersicht/

chrisdwheatley avatar Mar 06 '14 14:03 chrisdwheatley

@daonb That's actually a great idea!

espy avatar Mar 06 '14 14:03 espy

@daonb I just implemented your idea with the github.io subdomain, thanks a lot for that. Ubersicht will now use the gh subdomain, if it exists, and if no default organisation name is provided to override it.

Back to your pull request: thanks a lot for the work, but I'm afraid I cant merge it because clicking the weekly indicators you added still shows the results for the past 24 hours, and not for the past 7 days. Also, they don't behave responsively:

ubersicht_weekly_ipad

Sorry :/ But say, would it be useful to you if you could configure the time range used for the two indicators and the "last 24 hours only"-checkbox? So you could have them use 48 hours, or 7 days, or a month? Or do you want both the 24h and the 7 day indicators at once?

espy avatar Mar 06 '14 15:03 espy

I like the idea of showing the next bigger range if the current is 0 and hiding the 0 ones.

janl avatar Mar 06 '14 15:03 janl

Oooh, fancy. I like it.

espy avatar Mar 06 '14 15:03 espy

Sorry for the bugs. Should be fixed now.

daonb avatar Mar 12 '14 08:03 daonb

Interests

lycon2014 avatar Nov 13 '15 00:11 lycon2014