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

E1791. Track the time that students look at the other submissions --> logging improvement (latest version) #1124

Closed
wants to merge 23 commits into from

Conversation

jerry-shijieli
Copy link
Contributor

@jerry-shijieli jerry-shijieli commented Dec 6, 2017

  1. Transferred review submission link/file time logging code to the latest Expertiza repo;
  2. Refactored 'respone_time' model to 'submission_viewing_event';
  3. Set up 'submission_viewing_events' table to record starting and ending time of submission review;
  4. Installed d3 library in bower for review time data visualization;
  5. Changed Pie chart to Doughnut chart and added time distribution proportion on detailed submission review time visualization;
  6. Add real-time saving of review submission viewing time;
  7. Redesign popup detail page display content.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling cebdfcd on WillSuen:master into ** on expertiza:master**.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 61.71% when pulling 1227346 on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling 9afc724 on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling d53c543 on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling 7509a8a on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling 2f61ebd on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling 2f61ebd on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 61.71% when pulling fd4ca20 on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 61.69% when pulling 7aa1cc8 on WillSuen:master into 3d78df6 on expertiza:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 61.71% when pulling 7aa1cc8 on WillSuen:master into 3d78df6 on expertiza:master.

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