Skip to content

Make sure messages get jump_to_id URLs rewritten. - #64

Merged
Kelketek merged 1 commit into
masterfrom
feedback-jump-to-id
Sep 21, 2015
Merged

Make sure messages get jump_to_id URLs rewritten.#64
Kelketek merged 1 commit into
masterfrom
feedback-jump-to-id

Conversation

@Kelketek

Copy link
Copy Markdown
Member

@bradenmacdonald @e-kolpakov @antoviaque

To test this:

  1. Create an MCQ with a message sub-block that has a /jump_to_id/<id_hash> url link in it.
  2. Add in a message feedback block that also has one.
  3. Set the mode to assessment.
  4. Fail to answer the MCQ correctly, see the results page, check the links to see if they work.

@bradenmacdonald

Copy link
Copy Markdown
Member

👍

Kelketek added a commit that referenced this pull request Sep 21, 2015
Make sure messages get jump_to_id URLs rewritten.
@Kelketek
Kelketek merged commit be0c46a into master Sep 21, 2015
@Kelketek
Kelketek deleted the feedback-jump-to-id branch September 21, 2015 14:33
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.

2 participants