inveniosoftware / inveniosoftware/react-invenio-app-ils
frontsite: patron cancel modal
Open
refactor
- Dominant language
- JavaScript
- Stars
- 6
- Forks
- 24
- Avg merge
- 2d 46m
- Merged PRs (30d)
- 3
Description
As discussed in the review of https://github.com/inveniosoftware/invenio-app-ils/pull/759#discussion_r385564841, refactor the modal by introducing an intermediate component which will encapsulate the trigger button and the modal state.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing the patron cancel modal and the discussion in the linked pull request, especially the referenced review comment. Trace where the trigger button and modal state are currently handled. Done means an intermediate component encapsulates both responsibilities without changing the modal's behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100