-
Notifications
You must be signed in to change notification settings - Fork 25
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
No Sessions Available #1
Comments
Hmm, that's a good issue. Let me explain. There's a "year" filter, and a "month" filter. If no month or year is selected, it chooses the CURRENT month/year. The options in the drop down are populated from the possible values stored in the database. I started recording tracks in October 2014. So, you can choose 2014...but when you do, if you filter just on that, it's currently choosing April 2014 (since it's April 2016), where there's no tracks. The way I have the filtering done, I'm not sure how to fix that...I guess upon choosing a year, it could re-query the months (and vice versa) and try to dynamically update the month list...I'll have to look into that. |
…e month filter unless it's explicitly specified. Basically, it only chooses the current month if no filter is applied to date. Profile filter doesn't change anything, and choosing a month does NOT change the year behavior. I don't think it makes sense to say 'I chose April, so show me every possible April for as long as we've been recording tracks.'
Ok, I pushed out a POTENTIAL fix. Not sure if it's better or not. Commit is here: ALWAYS the session that you are filtering from will be included, even if it's outside of the filter parameters. This is to prevent a situation where no session loads. Cases: I don't know if that's better? Should reduce the "no sessions" instances, unless you choose a month in the future. |
Perfect, |
Allow set other mysql connection port
In the demo,
I noticed that you can filter to sessions in 2014 however no sessions exist in 2014.
It would make more sense to only filter by the range of dates that you do have.
The text was updated successfully, but these errors were encountered: