edehr / edehr/edehr

Clean-up getPageErrors in ehr-helper

Open
#1,079 0 comments 0 reactions 0 assignees View on GitHub
good first issue help wanted
Dominant language
JavaScript
Stars
14
Forks
11
PR merge metrics
No merged PRs in 30d

Description

In the ehr-helper file, we have a error-handling function which is no longer being used, you can see it on line 82 and it is currently just returning an empty list.

The task for this issue is a very simple one, you will need to remove the getPageErrors as well as cleaning-up the code where it's being used, including in ehr-helper tests.

This is a great first project for a first contributor, since its scope and impact are low, you will only need to look for references of places in the code using it (there are currently 2: one in EhrPageForm, one in ehr-helper.spec.js) as well as the actual function definition in ehr-helper.js).

Contributor guide

Open the contributing guide

Research direction

Start by inspecting ehr-helper.js around line 82 and search for the two references named in the issue: EhrPageForm and ehr-helper.spec.js. Remove getPageErrors and clean up those usages and the related test code. Done means no remaining references and the ehr-helper tests still pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.