Anton/allow arbitrary regex in stringresponse - #1948
Conversation
There was a problem hiding this comment.
Might as well fix "areintended" while you're here.... :)
There was a problem hiding this comment.
Fixed: Might as well fix "areintended" while you're here.... :)
|
@nedbat I've addressed all your comments. Please continue review. |
There was a problem hiding this comment.
use camelCase (firstAnswer instead first_answer).
|
@polesye all your comments are addressed. |
|
👍 |
There was a problem hiding this comment.
Don't add these four files, please. They are due to a known problem on master, but should not be added.
There was a problem hiding this comment.
They were added accidentally through negligence. Thanks, fixed: "Don't add these four files, please. "
|
I can finish the review when the semantics of the regex match have been finished. |
|
@nedbat All your comments are addressed, please continue review. |
|
👍 |
…ngresponse Anton/allow arbitrary regex in stringresponse
4 participants
This is for https://edx-wiki.atlassian.net/browse/BLD-475: Allow arbitrary regex inside of string response and updates for multiple answers for string response.
Add mode to markdown editor: if first answer is started with | -> that will be regexp.
@Lyla-Fischer @polesye @nedbat Please review.
@srpearce this PR contains documentation.