Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove courses and sessions from userService.getUserProfile and refactor module imports #721

Open
6 tasks
kyleecodes opened this issue Dec 7, 2024 · 0 comments
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days help-wanted Extra attention is needed refactor state: blocked Task is blocked.
Milestone

Comments

@kyleecodes
Copy link
Member

Overview

After implementing the new /courses_user endpoint (Issue #720 ) and updating the frontend to use it (frontend Issue #1223), we need to refactor the userService.getUserProfile function to no longer include courses and sessions data. This will reduce unnecessary data fetching and optimize the profile retrieval process. Additionally, we will need to clean up module imports to ensure that we aren't importing unnecessary modules or creating circular dependencies.

Blocked by #720 and this frontend issue: chaynHQ/bloom-frontend#1223

Action Items

  • Remove courses and sessions data from the userService.getUserProfile function to avoid pulling unnecessary data in user profile requests.
  • Refactor the service logic so that courses and sessions are no longer included by default when fetching the user profile.
  • Update module imports to prevent unnecessary circular dependencies and ensure the codebase remains modular.
  • Test the backend API to confirm that courses and sessions data are no longer being returned in the profile response.
  • Ensure that the new /courses_user endpoint is still functional and properly providing courses and sessions data for course and session pages.
  • Ensure no regressions occur in the application and that no unnecessary data is fetched when loading the user profile.

Resources

@kyleecodes kyleecodes added complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days help-wanted Extra attention is needed refactor state: blocked Task is blocked. labels Dec 7, 2024
@kyleecodes kyleecodes added this to the 02. Roadmaps milestone Dec 7, 2024
@kyleecodes kyleecodes changed the title Remove courses and sessions from userService.getUserProfile Remove courses and sessions from userService.getUserProfile and refactor module imports Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: advanced Time needed to do this ticket will be large e.g. 2-3+ days help-wanted Extra attention is needed refactor state: blocked Task is blocked.
Projects
None yet
Development

No branches or pull requests

1 participant