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 | 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 |
|
