[BD-03] Status Notes
Time Frame | Updates & Actions Needed | ~Hours |
---|---|---|
2020-04-06 - 2020-04-20 | Discoveries on available approaches for YellowDig and Piazza, two initial integrations being considered. Investigate SSO and LTI integration options for YellowDig and Piazza Update: Both support LTI integration. As a result the need for LTI embedding out of courses was added to the scope of this work. | 17 Total = 17/1200 |
2020-04-20 - 2020-05-04 | Discoveries to move some of the course apps out of the core platform were conducted. Discovery: Move Textbooks app to external plugin Discovery: Move WIki app to external plugin Update: The results of the discovery were that move these plugins out of the core, we need a standard API for external apps to store information in the course that will be automatically imported and exported, or we need a import-export API that can hook into the export process and include data from external apps, and hook into the import process to import data from external apps. | +11 Total 17+11 = 28/1200 2.3% - 5/04 |
2020-05-18 - 2020-06-01 | Discoveries to investigate a rough API boundary for Discussion Apps and for configuring such apps. Discovery: Configuration API Discovery: Discussion App API Update: A rough boundary between Discussion Apps and the core platform was investigated and discussed. A PR was created to capture this: https://github.com/edx/edx-platform/pull/24145 and it will be updated as these APIs are implemented concretely. | +18 Total 28+18 = 46/1200 3.8% - 6/01 |
2020-06-01 - 2020-06-15 | There were a number of tasks scheduled for this sprint: Extract LTI consumer code from XBlock to a library that cab be used by the platform [BD-02 BD-03 Add support for LTI embeds in course tabs and elsewhere by pcockwell · Pull Request #77 · openedx/xblock-lti-consumer ] Implement Configuration models from discovery above [https://github.com/edx/edx-platform/pull/24190 ] status:RejectedConsolidate the code that deals with cs_comments in one place, to create a boundary between the generic discussions code and the code specific to the cs_comments service. Update: The first task is now merged and included additional work from the next sprint. The second one is complete but is held back from further review to allow for refactoring from task 5 first. The final task turned out to be overly ambitious since it involves massive code changes on a moving target. It will now instead be done in small bits with other tasks. | +82 Total 46+82 = 128/1200 10.6% - 6/15 |
2020-06-15 - 2020-06-29 | There were a number of tasks scheduled for this sprint: status:SpilloverAdd a new field to courses to enable/disable discussions and add a similar field to verticals/units to enable discussions for that vertical/unit status:SpilloverStart laying the groundwork for the course-authoring MFE Add API to embed LTI content outside of the XBlock, and potentially outside of a course Updates: The first two task were close to complete but with final review comments to be addressed. With this PR, the third task here was completed along with the LTI task from the previous PR. | +68 Total 128+68 = 196/1200 16.3% - 6/29 |
2020-06-29 - 2020-07-13 | Since tasks were left over from the previous sprint, they continued in this one. status:SpilloverAdd a new field to courses to enable/disable discussions and add a similar field to verticals/units to enable discussions for that vertical/unit [https://github.com/edx/edx-platform/pull/24380 ] status:SpilloverStart laying the groundwork for the course-authoring MFE [[BD-03] [BB-2542] ]Add discussions configuration UI wireframe in MFE by rusrushal13 · Pull Request #2 · openedx/frontend-app-authoring ] Discussions forum MFE conversion [[WIP] [BB-2856] [BB-2747] [BB-2748] Topics, Posts, and Threads UI by pcockwell · Pull Request #1 · open-craft/frontend-app-discussions ] The leftover tasks form previous sprint continued, but they are still missing some polish before being ready for upstream review. A skeleton with basic API integration was created. It needs to be the moved to the edX org. | +67 Total 196+67= 263/1200 22% - 7/13 |
2020-07-13 - 2020-07-27 | With BD-29 ramping down, a lot of work was scheduled for this sprint: status:TIMEBOXED [TNL-7353] Create React-based forum post editor status:Review needed [TNL-7342] Add plugin support to current Discussions tab status:delayedstatus:Review needed [TNL-7348] Add discussion enable/disable status to course blocks, and outline API status:Review needed [TNL-7349] Studio UI to edit discussions toggle status:Review needed [TNL-7339] Convert xblock_handler API to support non-session based auth status:Review needed [TNL-7341] Create python API to map external and internal discussion ids status:Review needed [TNL-7347] Implement Discussions Plugin loading and listing API The aim this sprint is to get the platform to the point where you have a code-only route to enabling a third-party discussion tool in the discussion tab. Update: We are now at a point where a rough discussion tab integration is possible for an external plugin. This can be seen in this sandbox: https://pr24584.sandbox.opencraft.hosting/dashboard The Discussions tab for course course-v1:Test+TST101+2020 is now provided by a simple test plugin that just inject some basic HTML into that page. (Unfortunately since we’re in the process of testing a Mongo upgrade the forums for our sandboxes are broken. This isn’t a result of the plugin work, but we will try to fix that issue so that it doesn’t mask something else). The task to create a discussion post editor was restricted in this sprint to start looking for react-based editors that match the requirements. “Add discussion enable/disable status to course blocks, and outline API” was delayed since the original PR to add that field was only ready for review this sprint. | +95 Total 263+95= 358/1200 29.83% - 8/7 |
2020-07-27 - 2020-08-10 | Basic Plugin to embed Piazza in discussions tab using LTI status:Review needed [TNL-7353] Create React-based forum post editor status:Delayed Use Discussion ID Map in existing cs_comments_service code status:SpilloverDiscovery: Evaluate API changes required for existing discussions to enable functionality in wireframes Setting up a sandbox for Discussions here: https://discussions.sandbox.opencraft.hosting Will keep this updated with latest consolidated code. | +52 Total 358+52= 410/1200 34.16% - 8/12 |
2020-08-10 - 2020-08-24 | Discovery: Evaluate API changes required for existing discussions to enable functionality in wireframes status:IN internal review [TNL-7411] [Discussions MFE] Posts UI status:IN internal review [TNL-7412] [Discussions MFE] Threads UI status:IN internal review [TNL-7413] [Discussions MFE] Topics UI status:IN internal review[TNL-7414] [Discussions MFE] Discussions Javascript API Code [Authoring MFE] Base forum settings editor [Authoring MFE] Discovery: settings plugins In this sprint we made good progress in getting the new discussions UI into shape. It is currently in internal review and for the next sprint I plan to set up demo instance of the new frontend. There were issues getting a sandbox up due to the issue fixed in this unmerged PR. We are now using that as a basis for setting the demo instance. Unfortunately, there have been other major changes to the way plugins work in the platform, so the existing branches are broken. The base forum setting editor will need to be rethought a bit since the discussion in the last meeting. The UI is done, but it might need to be pushed to a different MFE than originally thought. | +80 Total 410+80 = 490/1200 40.83% - 8/25 |
2020-08-24 - 2020-09-07 | [TNL-7432] [edx-platform] Extract all course discussion settings into new API for MFEs [TNL-7431] [Authoring MFE] Implement forum tool selector wireframe Prototype: Implement frontend plugin loading mechanism Setup demo instance | +84 Total 490+84 = 574/1200 47.83 - 9/8 |
2020-09-07 - 2020-09-21 | [edx-platform] [cs_comments_service] Add Discussion APIs support new features in Discussions MFE - Part 1 [Discussions MFE] Add embeddable views to Discussions MFE [Discussions MFE] Add forum new post UI (without editor) | +53 Total 572+52/1200 = 627/1200 52.25 - 9/21
|
2020-09-21 - 2020-10-05 | @Kshitij Sobti Create ADR documents for all PRs | +35 Total 627+35/1200 = 662/1200 55.16 - 10/5
|
2020-10-05 - 2020-10-19 | @Kshitij Sobti Sync up status on current TNL tickets @Kshitij Sobti TNL-7347: Accommodate review feedback, simplify PR and get it merged
| +23 Total 662+23/1200 = 685/1200 57.00 - 10/20 |
2020-10-19 - 2020-11-02 | https://openedx.atlassian.net/browse/TNL-7616 If the Discussion Configuration PR merges we can begin work on this to allow its use from frontend-course-authoring Updates: The configuration system was rewritten to be simplified. The plugin interface was also cleaned up and rebased on the above configuration PR. There are some tests failing on the CI that aren't locally that @Kshitij Sobti is investigating. The frontend work was rebased on the latest changes, and will be merged after refreshing it based on the newer designs. | +20 Total 685+20/1200 = 705/1200 58.75 - 11/03 |
2020-11-03 - 2020-11-17 | Updates:
| +30 Total 705+30/1200 = 735/1200 61.25 |
2020-11-17 - 2020-12-01 |
| +7 Total 735+7/1200 = 742/1200 61.82 |
2020-12-01 - 2020-12-15 | LTI Tabs Discovery | +30 Total 742+30/1200 =772/1200 64.33 |
2020-12-15 - 2020-12-29 | LTI Course Tab Implementation LTI-based Discussion Course tab |
|