-
Notifications
You must be signed in to change notification settings - Fork 114
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
InlineQuestion calls markup_to_html without a PageContext #80
Comments
Yes, you're right. That will cause problems, for example, course specific links and jinja macros won't be rendered correctly. The reason I did that is, I need to use markup_to_html to validate / get question body and answer items (of each types) in I will try to fix it. |
I think I know how to fix it. Validation and page rendering should be separated: use vctx in validation, and use page_context when rendering html. Need sometime to modify and test. |
Should be fixed in #82. However, I found images and latex expressions are not rendered correctly in |
@dzhuang
I just discovered that you don't pass a
PageContext
tomarkup_to_html
in InlineQuestion, here for example:relate/course/page/inline.py
Line 382 in 882803a
As of 882803a, this breaks caching of rendered markup. (it has to, since markup rendering depends on the course for which it is done). Could you please fix this?
The text was updated successfully, but these errors were encountered: