openedx / openedx/frontend-app-admin-console
Review Error handling after refactoring
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 1
- Forks
- 13
- Avg merge
- 1d 9m
- Merged PRs (30d)
- 6
Description
Description
There are a few areas where the error messages could be improved. There is important to make an analysis of all the current places errors needs to be displayed and how they should be displayed (Toast, other element such as the Error Page or an Alert)
For example, when a user does not have access to any course or library (an edge case), the application displays a generic error message instead of indicating that the user lacks the required permissions or access.
It is also important to review whether the error page should be displayed and update the messages if needed (if they are generic we should consider move it inside the component instead of the module location)
Steps to Reproduce
- Log in with a user account that does not have access to any course or library.
- Navigate to the application.
- Observe the error message displayed.
Actual Behavior
A generic error message is displayed.
Expected Behavior
The application should display a more specific error message indicating that the user does not have access to any courses or libraries, rather than showing a generic error. This would provide clearer feedback and help users better understand the reason for the failure.
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 by reproducing the no-access case with a user who has no course or library access, then trace where the generic error is selected and displayed. Review the application’s error messages and decide whether each belongs in a Toast, alert, error page, or component. Done means the no-access case gives a specific permissions message and the relevant error-page behavior and wording are reviewed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- authorization, frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100