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.

Sign up for GitHub

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

Jump to bottom

Time picker for Coach Reports #4196

Merged
merged 2 commits into from
Aug 2, 2015
Merged

Time picker for Coach Reports #4196

merged 2 commits into from
Aug 2, 2015

Conversation

rtibbles
Copy link
Member

@rtibbles rtibbles commented Aug 1, 2015

Fixes fle-internal/ka-lite-central#269

Summary of changes:

  • Implements a time picker on the coach report landing page
  • Allows for the setting of arbitrary date ranges for summary view and tabular report.
  • Adds a dynamic setting to allow different default time windows.

@jamalex

@rtibbles rtibbles added the has PR label Aug 1, 2015
@rtibbles rtibbles added this to the 0.14.0 milestone Aug 2, 2015
@jamalex
Copy link
Member

jamalex commented Aug 2, 2015

This looks good to me. Let's get it merged in and get some testing on it.

jamalex added a commit that referenced this pull request Aug 2, 2015
@jamalex
Merge pull request #4196 from rtibbles/time_picker
aeb8b5c 
Time picker for Coach Reports
@jamalex jamalex merged commit aeb8b5c into learningequality:0.14.x Aug 2, 2015
@jamalex jamalex removed the has PR label Aug 2, 2015
@rtibbles rtibbles deleted the time_picker branch March 2, 2016 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.14.0
Development

Successfully merging this pull request may close these issues.
2 participants
@rtibbles @jamalex