Skip to content

Conversation

@KristinAoki
Copy link
Contributor

@KristinAoki KristinAoki commented Dec 11, 2023

Description

Depending on the amount of courses and libraries a user has access to, the home page can timeout while trying to fetch all the data in the Course Authoring MFE. This PR splits the home api into three parts:

  1. Basic functionality call (user access, library access, etc)
  2. Courses data call
  3. Libraries data call

On the initial load of the page the first two apis are called one after the other. The library api is called when the library tab is clicked. This behavior aligns more closely with the legacy api call behaviors. This change impacts Authors.

Supporting information

JIRA Ticket: TNL-11244

Users are seeing the page load. The load time is very long and after the loading spinner goes away the page does not load correctly. The API is throwing a 504 (Gateway timeout) error.

Testing instructions

Testing

  1. Open the Network tab in DevTools
  2. Navigate to the Studio home page
  3. Page should load as expected tabs and number of courses
  4. Look at the Network tab and confirm that the home api call did not include courses, archived courses, or libraries
  5. Look at the Network tab and confirm that the courses api call was successful
  6. Click on the Library tab
  7. Should show spinner while loading libraries
  8. After loading completes, expected libraries should be listed

Deadline

None

Other information

frontend-app-course-authoring PR #740 is dependent on the change.

@KristinAoki KristinAoki force-pushed the KristinAoki/update-studio-home-api branch from fbab6ea to f0f5ad3 Compare December 11, 2023 15:27
@KristinAoki KristinAoki changed the title feta: separate home api call into load, course, and library info feat: separate home api call into load, course, and library info Dec 11, 2023
@KristinAoki KristinAoki marked this pull request as ready for review December 11, 2023 21:44
Copy link
Contributor

@rayzhou-bit rayzhou-bit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@KristinAoki KristinAoki merged commit 9f5252c into master Dec 19, 2023
@KristinAoki KristinAoki deleted the KristinAoki/update-studio-home-api branch December 19, 2023 13:32
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants