openedx / openedx/frontend-app-admin-console

Review Error handling after refactoring

Open
#174 0 comments 0 reactions 0 assignees View on GitHub

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
  1. Log in with a user account that does not have access to any course or library.
  2. Navigate to the application.
  3. 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.

Image

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.