Code Review Plan
Pending & In-Progress PRs
|
|
|---|---|
Pending | edX Code Review not started yet. No blockers, the PR is waiting for its turn in queue or is a WIP. |
In Review | Actively under edX Code Review and feedback phase. |
Blocked | edX Code Review on hold. More details on PR. |
Approved | edX Code Review Done. PR got 2 from edX team but it is not yet merged. |
WIP | PR is a work in progress, but is included here so it can get early feedback. |
Opencraft / infinity | Status to further explain out the codes above with regards to the team weather its open-craft or infinity. |
No. | Status | Reason (if any) | Milestone | PR# | Description | Reviewer |
1 | in review | ADR for information only | v1.7 | ADR about topic structures |
|
Completed
133
No. | Milestone | PR# | Description |
|---|---|---|---|
1 | v1.3 |
| |
2 | v1.3 | v1.3 Moves the | |
3 | v1.5 | edx-platform#27542 feat: Course Apps API [BD-38] [TNL-8103] [BB-2716] | Implementation of Course Apps API |
4 | v1.5 | edx-platform#27193 docs: ADR for new course apps API [BD-38] [TNL-8002] [BB-3964] | ADR that introduces a new type of plugin called a “course app” which will show up in the new pages and resources API. |
5 | v1.3 | v1.3 The other half of the above PR. It is dependent on the above PR since the version bump in this PR will be for the above. | |
6 | v1.5 | https://github.com/edx/frontend-app-course-authoring/pull/63 | v1.5 Implements the individual components of the pages and resources view, and hooks them up to the new APIs above and below. |
7 | v1.4 | edx-platform#26982 feat: Allow optionally passing PII in course LTI tab | Allows optionally passing the PII in the discussions LTI tab if the above flag is set. |
8 | v1.4 | BE: kewne: Disable changing discussions providers - edx-platform/pull/27825 | Disables changing discussion providers once a course run has started unless the user is a global staff. |
9 | v1.4 | v1.3 Second phase of refactoring the PII Sharing flag | |
10 | v.16 | feat: adds additional data and filters to discussions API. cs_comments_service#345 | Adds additional filters and data to the |
11 |
| Rejected | |
12 | v.1.6 | feat: Adds additional data and filters to discussions API edx-platform#27914 Depends upon cs_comments_service#345 | Adds new data and filters to the discussions API in |
13 | v1.6 | FE: Add topic listing views, post listing view, comment views: frontend-app-discussions/pull/8 | Discussions MFE first iteration of UI |
14 | v1.5 | Backend change for TNL-8438 | |
15 | v1.3 | Backend change for TNL-8339 | |
16 | v1.3 | https://github.com/edx/frontend-app-course-authoring/pull/154 | Frontend change for TNL-8339 |
17 | v1.5 | https://github.com/edx/frontend-app-course-authoring/pull/156 | Frontend change for TNL-8438 |
18 | v1.5 | edx-platform#27608 feat: Add REST APIs for course advanced settings and course tabs [BD-38] | Exposes the course advanced settings API and the course tabs API, so they can be accessed through MFEs. |
19 | v1.6 | Adds support for searching all posts to discussions MFE | |
20 | v1.4 | https://github.com/edx/frontend-app-course-authoring/pull/171 | UI to enable/disable PII sharing in MFE |
21 | v1.3 | Adds flag to disable changing LTI config for YellowDig (and potentially other providers). | |
22 | v1.4 | Return course apps learn more URLs from the backend | |
23 | v1.4 | https://github.com/edx/frontend-app-course-authoring/pull/173 | Frontend change for above |
24 | v1.3 | https://github.com/edx/frontend-app-course-authoring/pull/176 | Frontend change for above. Adds message if users can’t edit LTI config, and hides LTI edit UI for them. |
25 | v1.6 | Brings the MFE closer to mockups. Switches fonts etc. | |
26 | v1.4 | FE: kewne: Disable changing discussions providers: frontend-app-course-authoring/pull/126 | frontend component of above PR. |
27 | v1.6 | Frontend - Adds the Add new Post UI for discussions. | |
28 | v1.4 | Gives access to course advanced settings allowing course apps like progress to update settings using an API. | |
29 | v1.4 | https://github.com/edx/frontend-app-course-authoring/pull/151 | Frontend for editing settings for notes, progress, and wiki |
30 | v1.6 | Small change to include staff / moderator user markers to UI | |
31 | v1.6 | Mark posts as read automatically on view after a delay | |
32 | v1.6 | Improve URL routing and paths | |
33 | v1.4 | Add custom pages card. | |
34 | v1.4 | Backend - Adds a data model for course apps API | |
35 | v.1.7 | Backend Doc - ADR for in-context discussions | |
36 | v1.6 | Just updates dependencies and updates some of the code to match frontend template application | |
37 | v1.6 | Use TinyMCE editor | |
38 | v1.6 | Adds tests for the data layer, moves the normalisation to the thunk etc. | |
39 | v1.7 | ADR for how to access discussion embed links from MFE. | |
40 | v1.4 | https://github.com/edx/frontend-app-course-authoring/pull/187 | UI to edit team sets |
41 | v1.6 | Adds a new dropdown for posts and comments that automatically populates valid actions based on permissions and the available editable fields. | |
42 | v1.6 | Replaces custom sanitising logic with the bleach library for XSS protection | |
43 | v1.6 | Adds editor for replies | |
44 | v1.6 | Add support for paginating comments in a post/thread. | |
45 | v1.4 | Make wiki configurable | |
46 | v1.6 | Add support for certain operation currently not allowed by the discussions REST API. | |
47 | v1.4 | Make it possible to enable/disable the wiki | |
48 | v1.4 | https://github.com/edx/frontend-app-course-authoring/pull/193 | Implements feedback from TNL-8730 |
49 | v1.6 | Add support for questions | |
50 | v1.6 | Add support for paginating comments in comments. | |
51 | v1.5 | https://github.com/edx/frontend-app-course-authoring/pull/201 | Teams page styling fixes |
52 | v1.6 | Adds support for loading posts on scroll. | |
53 | v1.4 | Updated design of custom pages, and only include custom tabs in UI. | |
54 | v1.6 | Add support for returning thread counts for topics to cs_comments_service | |
55 | v1.5 | Fix for TNL-8820 | |
56 | v1.6 | Uses above PR to adds support for thread counts to edx-platform APIs. | |
57 | v1.4 | ADR for mechanism to enable/disable teams | |
58 | v1.5 | https://github.com/edx/frontend-app-course-authoring/pull/205 | Teams setting UX fixes #3 |
59 | v1.5 | Fixes the notes tab not showing up. | |
60 | v1.7 | Create new discsussions settings for the new experience. Store discussions settings in course. Refactor the discussion settings API to work from studio and modify modulestore content. | |
61 | v1.6 | Implements new filter bar and updates UX to bring it closer to new mockups | |
62 | v1.7 | https://github.com/edx/frontend-app-course-authoring/pull/197 | Frontend change for https://github.com/edx/edx-platform/pull/28749 above |
63 | v1.5 | Implements “ADR for mechanism to enable/disable teams” | |
64 | v1.6 | Add support for getting anonymous post settings for existing posts and new ones. | |
65 | v1.6 | Add the ability to select a cohort when posting a comment | |
66 | v1.6 | Tests for action dropdown | |
67 | v1.6 | Tests for home component | |
68 | v1.6 | Adds a new image upload endpoint for discussions for use with TinyMCE. | |
69 | v1.6 | TinyMCE improvements | |
70 | v1.6 | Tests for topics | |
71 | v1.7 | Adds a new field to units to mark them as discusssable. Will be used in case of custom visibility. | |
72 | v1.6 | Show cohort info in posts UI | |
73 | v1.5 | https://github.com/edx/frontend-app-course-authoring/pull/210 | Switches view live button to use learning mfe link. |
74 | v1.7 | Add support for hooks and signals to maintin discussoin topic links in the database. | |
75 | v1.7 | Hide the UI to add a discussion block if the course is not using the legacy provider | |
76 | v1.6 | Hooks up the anonymous posting support | |
77 | v1.6 | Add topic link to course structure | |
78 | v1.5 | Refactors and updes the tab ordering code. | |
79 | v1.6 | Add a waffle flag to show discussions MFE in-context and link the Discussions tab to it. | |
80 | v1.6 | Adds new breadcrumb bar | |
81 | v1.9 v1.10 | Add a new API that returns user stats. I asked for early fedback on this and was suggested by Dave Ormsbee that we should test this in a small way in produciton and check the performance that way since evaluating the performance any other way would be hard. | |
82 | v1.7 | Add support for smaller screens and in-context embedded views. Needs tests, and will need to be updated for the breadcrumbs work but is otherwise done. | |
83 | v1.6 | Added a user privilige and role infomration to dicussions API. | |
84 | v1.7 | Adds discussion context to course blocks API | |
85 | v1.6 | Completes supports from cohorts | |
86 | v1.6 | Fix autosaving | |
87 | v1.9 v1.10 | Adds APIs to cs comments service to get comments filtered by user id and course id instead of thread. | |
88 | v1.6 | Fixes the topic counts in the topics page | |
89 | v1.6 | Redirects to the all posts page when searching posts | |
90 | v1.6 | Fixes some of the redirects that broke when swichint to discussionPath | |