Skip to content
Snippets Groups Projects
Commit c5e2c87f authored by Bogdan Cochior's avatar Bogdan Cochior
Browse files

Merge branch 'HIN-1243-Fix-For-EIC-Admin' into 'develop'

fix(ManuscriptDecision): Changes applies to EiC and Admin too

See merge request !213
parents 5fe97fe0 d2ccab36
No related branches found
No related tags found
3 merge requests!222Sprint #26,!217Sprint #26,!213fix(ManuscriptDecision): Changes applies to EiC and Admin too
......@@ -155,6 +155,7 @@ export default compose(
values,
dispatch,
{
reset,
submitDecision,
showModal,
setFetching,
......@@ -164,6 +165,7 @@ export default compose(
},
) => {
showModal({
reset,
title: `${modalTitle}?`,
confirmText: modalTitle,
subtitle: modalSubtitle,
......
......@@ -11,22 +11,23 @@ export default compose(
fragment,
collection,
fetchUpdatedCollection,
}) => (values, modalProps) => {
}) => (values, { setFetching, setModalError, hideModal, reset }) => {
const recommendation = parseEicDecision(values)
modalProps.setFetching(true)
setFetching(true)
createRecommendation({
recommendation,
fragmentId: fragment.id,
collectionId: collection.id,
})
.then(() => {
modalProps.setFetching(false)
modalProps.hideModal()
setFetching(false)
hideModal()
reset()
fetchUpdatedCollection()
})
.catch(err => {
modalProps.setFetching(false)
handleError(modalProps.setModalError)(err)
setFetching(false)
handleError(setModalError)(err)
})
},
onEditorialRecommendation: ({
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment