inveniosoftware / inveniosoftware/react-invenio-app-ils

Change action functions to be controlled in actions.js

Open
#32 1 comment 0 reactions 0 assignees View on GitHub
refactor
Dominant language
JavaScript
Stars
6
Forks
24
Avg merge
2d 46m
Merged PRs (30d)
3

Description

We are passing the actions in many components as a prop, and triggering them in the component itself.
The problem is mainly visible when we trigger the reloading of **another** component to include the `ES_DELAY` for indexing
This should be done on the actions level, not component level, as shown in
https://github.com/inveniosoftware/invenio-app-ils/blob/d2b1ab3310683c3b9ed81bf7d23f509eade47fd3/ui/src/pages/backoffice/Patron/PatronDetails/components/ItemsCheckout/state/actions.js
It should be refactored where applicable.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.