-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Doc - First draft for shuffle changes to Building Course guide #2550
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
Conversation
|
@nparlante @cahrens first draft of shuffle update is at: http://draft-shuffle.readthedocs.org/en/latest/common_problems.html#shuffle-answers-in-a-multiple-choice-problem I did this without actually running it, couldn't build a sandbox off of Nick's branch, caused errors, but wanted to get started. Will test when feasible. |
|
@lamagnifica @srpearce can you look over new section in link above when you get a chance. this isn't in tomorrow's release, possibly next week. |
|
Looks great--except for the "you must edit the XML in the _The Advanced On Tue, Feb 11, 2014 at 10:16 AM, Mark Hoeber notifications@github.comwrote:
|
|
Hey Mark -- note that there is a markdown syntax for the shuffle On 2/10/14 2:18 PM, Mark Hoeber wrote:
|
|
All, updated with markup for shuffle, and xml details for targeted feedback and answer pools. Can you give it one more look over. Won't be merging until code is released on edx.org. Thanks |
|
Here is some feedback I gave Mark through HipChat: I was reading through your documentation on Shuffle and Targeted Feedback before looking at the PR. DId your question (DO YOU STILL NEED SOLUTION ELEMENT) get addressed? http://draft-shuffle.readthedocs.org/en/latest/common_problems.html#shuffle-answers-in-a-multiple-ch... Also, it looks like the "Answer Pools" section has some copy/paste errors (11.3.4.1) And here, I would say "You do still use the instead of also For targeted feedback, your example code does not work. Your answer pool example doesn't work either. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be "to configure answer pools"?
|
Comments addressed, all updated, rebased. Shuffle: http://draft-shuffle.readthedocs.org/en/latest/common_problems.html#multiple-choice Targeted Feedback: http://draft-shuffle.readthedocs.org/en/latest/common_problems.html#targeted-feedback-in-a-multiple-choice-problem Answer Pools: http://draft-shuffle.readthedocs.org/en/latest/common_problems.html#answer-pools-in-a-multiple-choice-problem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo-- feeback3.
|
@mhoeber Looks like this branch needs a rebase. Also, do you know when it's going to be merged? |
|
@singingwolfboy Yes, will rebase it today. merging it depends on the standford shuffle feature being merged, which was postponed again. |
|
Blocked by #1499 |
…lding Course guide
Doc - First draft for shuffle changes to Building Course guide
No description provided.