Skip to content

E
edx-platform
Commit ac8c5912 by Victor Shnayder Committed by Victor Shnayder
Options

Add note about non-unique topic id across course runs bug

parent 0d41a044
Showing with 3 additions and 0 deletions
lms/djangoapps/django_comment_client/utils.py
...@@ -207,6 +207,9 @@ def initialize_discussion_info(course): ...@@ -207,6 +207,9 @@ def initialize_discussion_info(course):
"sort_key": entry["sort_key"], "sort_key": entry["sort_key"],
"start_date": entry["start_date"]} "start_date": entry["start_date"]}
# TODO. BUG! : course location is not unique across multiple course runs!
# (I think Kevin already noticed this) Need to send course_id with requests, store it
# in the backend.
default_topics = {'General': {'id' :course.location.html_id()}} default_topics = {'General': {'id' :course.location.html_id()}}
discussion_topics = course.metadata.get('discussion_topics', default_topics) discussion_topics = course.metadata.get('discussion_topics', default_topics)
for topic, entry in discussion_topics.items(): for topic, entry in discussion_topics.items():
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment