{"action":"created","issue":{"url":"https://api.github.com/repos/edx/edx-platform/issues/14862","repository_url":"https://api.github.com/repos/edx/edx-platform","labels_url":"https://api.github.com/repos/edx/edx-platform/issues/14862/labels{/name}","comments_url":"https://api.github.com/repos/edx/edx-platform/issues/14862/comments","events_url":"https://api.github.com/repos/edx/edx-platform/issues/14862/events","html_url":"https://github.com/edx/edx-platform/pull/14862","id":221082583,"number":14862,"title":"Default discussion topic","user":{"login":"replaceafill","id":560781,"avatar_url":"https://avatars1.githubusercontent.com/u/560781?v=3","gravatar_id":"","url":"https://api.github.com/users/replaceafill","html_url":"https://github.com/replaceafill","followers_url":"https://api.github.com/users/replaceafill/followers","following_url":"https://api.github.com/users/replaceafill/following{/other_user}","gists_url":"https://api.github.com/users/replaceafill/gists{/gist_id}","starred_url":"https://api.github.com/users/replaceafill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/replaceafill/subscriptions","organizations_url":"https://api.github.com/users/replaceafill/orgs","repos_url":"https://api.github.com/users/replaceafill/repos","events_url":"https://api.github.com/users/replaceafill/events{/privacy}","received_events_url":"https://api.github.com/users/replaceafill/received_events","type":"User","site_admin":false},"labels":[{"id":128413104,"url":"https://api.github.com/repos/edx/edx-platform/labels/needs%20triage","name":"needs triage","color":"e11d21","default":false},{"id":78051711,"url":"https://api.github.com/repos/edx/edx-platform/labels/open-source-contribution","name":"open-source-contribution","color":"bfe5bf","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-04-11T21:14:21Z","updated_at":"2017-04-17T17:58:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/14862","html_url":"https://github.com/edx/edx-platform/pull/14862","diff_url":"https://github.com/edx/edx-platform/pull/14862.diff","patch_url":"https://github.com/edx/edx-platform/pull/14862.patch"},"body":"This PR adds a course setting that allows you to configure the default topic for new posts in the Discussion tab.\r\n\r\n**Dependencies**: None\r\n\r\n**Screenshots**:\r\n\r\n![captura de pantalla de 2017-04-11 14-58-20](https://cloud.githubusercontent.com/assets/560781/24931100/35832a6a-1ec9-11e7-818b-464e6307c3c0.png)\r\n![captura de pantalla de 2017-04-11 14-59-49](https://cloud.githubusercontent.com/assets/560781/24931109/3ab4e0dc-1ec9-11e7-9a9b-49305baac338.png)\r\n\r\n**Sandbox URL**:\r\n\r\nLMS: https://oc2476.sandbox.opencraft.hosting/\r\nStudio: https://studio-oc2476.sandbox.opencraft.hosting/\r\n\r\n**Partner information**: 3rd party-hosted open edX instance\r\n\r\n**Deployment targets**: edx.org and edge.edx.org\r\n\r\n**Merge deadline**: None\r\n\r\n**Testing instructions**:\r\n\r\n1. In Studio go to the demo course advanced settings.\r\n2. Add a new category to the Discussion Topic Mapping. For example:\r\n\r\n```json\r\n\"Custom\": {\r\n    \"id\": \"custom\"\r\n}\r\n```\r\n\r\n3. Set the Discussion Default Topic ID to \"custom\" and save the changes\r\n4. In the LMS go to the demo course Discussion tab.\r\n5. Add a new post and check that the Topic area default value matches the setting.\r\n\r\n**Author notes and concerns**:\r\n\r\nWe tried to add a JS spec test for the DiscussionBoardFactory change, but the DiscussionRouter interferes with the history of other spec tests. That's the reason why [the current test for the factory is disabled](https://github.com/replaceafill/edx-platform/blob/bfbc3241b396ec9591674983c176839cb6f46aeb/lms/djangoapps/discussion/static/discussion/js/spec/discussion_board_factory_spec.js#L66). \r\n\r\n**Reviewers**\r\n- [ ] @pomegranited \r\n- [ ] @bdero "},"comment":{"url":"https://api.github.com/repos/edx/edx-platform/issues/comments/294543356","html_url":"https://github.com/edx/edx-platform/pull/14862#issuecomment-294543356","issue_url":"https://api.github.com/repos/edx/edx-platform/issues/14862","id":294543356,"user":{"login":"bradenmacdonald","id":945577,"avatar_url":"https://avatars1.githubusercontent.com/u/945577?v=3","gravatar_id":"","url":"https://api.github.com/users/bradenmacdonald","html_url":"https://github.com/bradenmacdonald","followers_url":"https://api.github.com/users/bradenmacdonald/followers","following_url":"https://api.github.com/users/bradenmacdonald/following{/other_user}","gists_url":"https://api.github.com/users/bradenmacdonald/gists{/gist_id}","starred_url":"https://api.github.com/users/bradenmacdonald/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradenmacdonald/subscriptions","organizations_url":"https://api.github.com/users/bradenmacdonald/orgs","repos_url":"https://api.github.com/users/bradenmacdonald/repos","events_url":"https://api.github.com/users/bradenmacdonald/events{/privacy}","received_events_url":"https://api.github.com/users/bradenmacdonald/received_events","type":"User","site_admin":false},"created_at":"2017-04-17T17:58:34Z","updated_at":"2017-04-17T17:58:34Z","body":"@gsong Yes. He's currently a candidate doing some work for us during a trial period, so it's under our CLA."}}
{"actor":{"display_login":"bradenmacdonald"}}