openedx / openedx/frontend-app-authoring
All courses being fetched in course outline page.
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 17
- Forks
- 218
- Avg merge
- 9d 20h
- Merged PRs (30d)
- 20
Description
When you go to a course outline, open network tab in the Developer console you will see a request like this /api/contentstore/v1/home/courses which fetches all courses in one instance.
I don't know if this data is being used for anything, but even if it is, it is still a major performance issue.
The code and the file are below:
useEffect(() => { dispatch(fetchStudioHomeData()); }, []);
https://github.com/openedx/frontend-app-authoring/blob/master/src/CourseAuthoringPage.jsx
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in src/CourseAuthoringPage.jsx at the useEffect that dispatches fetchStudioHomeData(), then trace whether the course list is used on the course outline page. Confirm the intended behavior with the existing page flow and verify that opening a course outline no longer requests /api/contentstore/v1/home/courses unnecessarily while the page still works.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend, performance
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100