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

Learner stats: Wrong login count and session tracking #5157

Closed
benjaoming opened this Issue Jun 13, 2016 · 5 comments

Comments

Projects
None yet
4 participants
@benjaoming
Member

benjaoming commented Jun 13, 2016

Summary

Received the following report:

After running the same tests as I had in IE, Chrome appears to be having the same issues, including no Student/Coach/Group Login and Login Times being recorded.

Below is a screenshot showing that students who have watched videos and completed exercises still do not have a Login of Login Time.

image001

I can confirm this, using our latest 0.16.x branch and a clean installation.

Branch or installer method

  • Branch: 0.16.x

How to reproduce

  1. Create a learner account
  2. Login
  3. Solve exercise
  4. Login as manager
  5. Go to facility
  6. See that there is no login count for the learner.

@benjaoming benjaoming added the bug label Jun 13, 2016

@benjaoming benjaoming added this to the 0.16.6 milestone Jun 13, 2016

@jamalex

This comment has been minimized.

Show comment
Hide comment
@jamalex

jamalex Jun 13, 2016

Member

👍 on this, from a report from Liberia today; this probably shouldn't hold back the 0.16.6 release (since that's coming tomorrow), but is an important bug to address soon.

Member

jamalex commented Jun 13, 2016

👍 on this, from a report from Liberia today; this probably shouldn't hold back the 0.16.6 release (since that's coming tomorrow), but is an important bug to address soon.

@j-schwartz

This comment has been minimized.

Show comment
Hide comment
@j-schwartz

j-schwartz Jul 1, 2016

Hey @benjaoming - I know we lose you soon, if we haven't already. Just curious if you think you'll be able to attack this prior to taking off. Any idea?

j-schwartz commented Jul 1, 2016

Hey @benjaoming - I know we lose you soon, if we haven't already. Just curious if you think you'll be able to attack this prior to taking off. Any idea?

@radinamatic

This comment has been minimized.

Show comment
Hide comment
@radinamatic

radinamatic Jul 1, 2016

Contributor

@j-schwartz, we lost @benjaoming already, at least until the end of the month... 😛

#5180 has already been merged, so you can check out the repo directly, try the 0.16.7 debian pre-release on PPA, or the latest Win build in Dungeon!

Contributor

radinamatic commented Jul 1, 2016

@j-schwartz, we lost @benjaoming already, at least until the end of the month... 😛

#5180 has already been merged, so you can check out the repo directly, try the 0.16.7 debian pre-release on PPA, or the latest Win build in Dungeon!

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Jul 27, 2016

Member

@j-schwartz -- this will be included in 0.16.7 as the PR is already merged, shall close this issue now :)

Member

benjaoming commented Jul 27, 2016

@j-schwartz -- this will be included in 0.16.7 as the PR is already merged, shall close this issue now :)

@benjaoming benjaoming closed this Jul 27, 2016

@j-schwartz

This comment has been minimized.

Show comment
Hide comment
@j-schwartz

j-schwartz Jul 27, 2016

Fantastic, many thanks Ben

On Wed, Jul 27, 2016 at 4:25 PM Benjamin Bach notifications@github.com
wrote:

@j-schwartz https://github.com/j-schwartz -- this will be included in
0.16.7 as the PR is already merged, shall close this issue now :)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5157 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI10dD25K8ZBVrrpu6MA5AxVt6bU1gIZks5qZ-jzgaJpZM4I0agk
.

j-schwartz commented Jul 27, 2016

Fantastic, many thanks Ben

On Wed, Jul 27, 2016 at 4:25 PM Benjamin Bach notifications@github.com
wrote:

@j-schwartz https://github.com/j-schwartz -- this will be included in
0.16.7 as the PR is already merged, shall close this issue now :)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#5157 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI10dD25K8ZBVrrpu6MA5AxVt6bU1gIZks5qZ-jzgaJpZM4I0agk
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment