Skip to content

Conversation

@anuragkanungo
Copy link
Contributor

Fixes #2947

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@anuragkanungo
Copy link
Contributor Author

Atleast for the entries we are sure about unit, let assign them the unit variable and keep others as 0.

If the unit variable is 101-104 then its displaying the correct unit as per our prediction.

But if the unit variable is 0 then we are not sure about the unit, 99% chances are that the unit is the current_unit.

As we can't predict the current unit on the central server, its better to have the value as 0 and let the coaches know that it is most probably the current unit number.

@jamalex
Copy link
Member

jamalex commented Feb 12, 2015

This should work for the majority of needed cases, so let's get it merged for now, and we can refine later.

jamalex added a commit that referenced this pull request Feb 12, 2015
Added Unit Variable to ExerciseLog Export
@jamalex jamalex merged commit 61fb56b into learningequality:nalanda-rct3 Feb 12, 2015
@jamalex jamalex removed the has PR label Feb 12, 2015
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.

3 participants