search

Stanford-Online / edx-platform

The Open edX platform, the software that powers Lagunita!
https://lagunita.stanford.edu
GNU Affero General Public License v3.0
42 stars 18 forks source link

Remove anonymous users from subsection view count #825

Closed tkim135 closed 6 years ago

tkim135 commented 6 years ago

@dcadams @stvstnfrd Management command edited to update subsection view count in instructor dashboard to exclude anonymous users.

For example, in the picture below, before the fix it used to say 2 students opened the subsection, and a username like "anon__h8104ht4g0rn" would appear. screen shot 2018-08-29 at 11 31 15 am

CC: @saharm1

dcadams commented 6 years ago

Looks like this is going in the right direction. You could take a look at the tests and see if any should be affected by this change.

tkim135 commented 6 years ago

@dcadams looks like it's a matter of migrations again? screen shot 2018-08-30 at 10 51 58 am

dcadams commented 6 years ago

I saw that. Is your branch up-to-date with master?

stvstnfrd commented 6 years ago

@dcadams Is this branch worth holding on to? If so, do you intend on running with it?