Teacher needs the server to support "retake"

[truedat101]

There is a new message for restart. We need to pass this through and change game state, which involves keeping all the students, keeping the questions, removing the score data, and put the game into start answering questions state.

[truedat101]

This is a child of: https://github.com/SMILEConsortium/smileplug-bsp/issues/9

[truedat101]

This is high priority.

[truedat101]

I need to provide a link to the source with the new Command Protocol that exists on the SMILE Student and Teacher Desktop repositories.

[truedat101]

Student source on adhoc branch: https://github.com/SMILEConsortium/smile_teacher_desktop/tree/adhoc should contain details of the command protocol for retake.

[truedat101]

Teacher java desktop source on adhoc branch: https://github.com/SMILEConsortium/smile_teacher_desktop/tree/adhoc

[pinazo]

As I could see in the smile_teacher_desktop code, server needs to support a message called RE_TAKEn, where n is the sequence of retake. As far as I can see, we don't need to take control of the number of retakes, is that correct?

[truedat101]

Hmm, I'm not sure. I'll need to run the teacher desktop app.    I think your assumption is correct.  Sent from my BlackBerry 10 smartphone. From: Juan PinazoSent: Saturday, May 18, 2013 2:36 PMTo: SMILEConsortium/node-smile-serverReply To: SMILEConsortium/node-smile-serverCc: truedat101Subject: Re: [node-smile-server] Teacher needs the server to support "retake" (#15)As I could see in the smile_teacher_desktop code, server needs to support a message called RE_TAKEn, where n is the sequence of retake. As far as I can see, we don't need to take control of the number of retakes, is that correct?

—Reply to this email directly or view it on GitHub.

[truedat101]

We may need to explore this issue later depending on how things work out with the RETAKEn n value.

[truedat101]

Just adding a comment for Thiago so he can review for what needs to be done for teacher: https://github.com/SMILEConsortium/smile_teacher_desktop/blob/adhoc/src/org/smilec/smile/teacher/HttpMsgForTeacher.java#L308

[truedat101]

I'm going to re-open. This probably needs some more code to interoperate with the teacher admin app. Also, appears to be missing a RE_TAKE test case.

[truedat101]

Confirmed the teacher desktop app sends a number along with the RE_TAKE message. The message structure seems to follow this:

• Teacher sends TYPE: RE_TAKEn as a POST to route: /JunctionsServerExecution/pushmsg.php
• Server receives a the post, handles thisz via index.handlePushMessage()
• Call game.setCurrentMessage() and then game.retake().
• Teacher app needs to:
  • increment the number of retakes. I'm not clear why this matters. Where is this ever used other than to pass it into the message that goes out
  • save the previous results automatically (we don't currently do this yet)
  • clean up any temporary data/state (not sure we have any)
  • reset ranking
  • reset the answers results
• The student picks up the message for retake: CourseList.RE_TAKE

[truedat101]

Student app on dev branch seems to be correct: https://github.com/SMILEConsortium/smile_student_android/blob/dev/src/org/smilec/smile/student/HttpMsgForStudent.java#L396

in that it will respond correctly to a retake.

[truedat101]

I'll close this as soon as I can confirm from that I can trigger the right sequence from the teacher app.