Skip to content
Snippets Groups Projects
Commit 3093cd33 authored by Anca Ursachi's avatar Anca Ursachi
Browse files

fix(manuscriptPage): Your Editorial Decision/Recommendation box, Your report...

fix(manuscriptPage): Your Editorial Decision/Recommendation box, Your report box, Abstract, Conflict
parent a7f6d099
No related branches found
No related tags found
3 merge requests!222Sprint #26,!217Sprint #26,!199fix(manuscriptPage): Your Editorial Decision/Recommendation box, Your report…
...@@ -79,11 +79,13 @@ const HERecommendation = ({ ...@@ -79,11 +79,13 @@ const HERecommendation = ({
handleSubmit, handleSubmit,
formValues, formValues,
highlight, highlight,
...rest
}) => ( }) => (
<ContextualBox <ContextualBox
highlight={highlight} expanded
label="Your Editorial Recommendation" label="Your Editorial Recommendation"
mb={2} mb={2}
{...rest}
> >
<Root> <Root>
<Row justify="flex-start"> <Row justify="flex-start">
......
...@@ -16,14 +16,22 @@ const ManuscriptMetadata = ({ ...@@ -16,14 +16,22 @@ const ManuscriptMetadata = ({
getSignedUrl, getSignedUrl,
currentUser: { token }, currentUser: { token },
fragment: { files = {}, conflicts = {}, metadata: { abstract = '' } }, fragment: { files = {}, conflicts = {}, metadata: { abstract = '' } },
abstractMetadataExpanded,
toggleAbstractMetadata,
toggleConflictsOfInterest,
conflictsOfInterestExpanded,
filesMetadataExpanded,
toggleFilesMetadata,
}) => ( }) => (
<Fragment> <Fragment>
{!!abstract && ( {!!abstract && (
<Item mb={1}> <Item mb={1}>
<ContextualBox <ContextualBox
data-test-id="abstract-tab" data-test-id="abstract-tab"
expanded={abstractMetadataExpanded}
label="Abstract" label="Abstract"
startExpanded startExpanded
toggle={toggleAbstractMetadata}
transparent transparent
> >
<Text mb={1} mt={1}> <Text mb={1} mt={1}>
...@@ -36,7 +44,9 @@ const ManuscriptMetadata = ({ ...@@ -36,7 +44,9 @@ const ManuscriptMetadata = ({
<Item mb={1}> <Item mb={1}>
<ContextualBox <ContextualBox
data-test-id="conflict-of-interest-tab" data-test-id="conflict-of-interest-tab"
expanded={conflictsOfInterestExpanded}
label="Conflict of Interest" label="Conflict of Interest"
toggle={toggleConflictsOfInterest}
transparent transparent
> >
<Row alignItems="center" justify="flex-start"> <Row alignItems="center" justify="flex-start">
...@@ -68,7 +78,13 @@ const ManuscriptMetadata = ({ ...@@ -68,7 +78,13 @@ const ManuscriptMetadata = ({
)} )}
{!isEmpty(files) && ( {!isEmpty(files) && (
<Item mb={1}> <Item mb={1}>
<ContextualBox data-test-id="files-tab" label={filesLabel} transparent> <ContextualBox
data-test-id="files-tab"
expanded={filesMetadataExpanded}
label={filesLabel}
toggle={toggleFilesMetadata}
transparent
>
<ManuscriptFileList <ManuscriptFileList
files={files} files={files}
getSignedUrl={getSignedUrl} getSignedUrl={getSignedUrl}
......
...@@ -55,15 +55,27 @@ const ManuscriptLayout = ({ ...@@ -55,15 +55,27 @@ const ManuscriptLayout = ({
handlingEditors, handlingEditors,
canHEOnlyReject, canHEOnlyReject,
toggleHEResponse, toggleHEResponse,
toggleEicDecision,
heResponseExpanded, heResponseExpanded,
toggleReviewReport,
eicDecisionExpanded,
reviewReportExpanded,
inviteHandlingEditor, inviteHandlingEditor,
toggleReviewerDetails, toggleReviewerDetails,
isFetchingFromAutosave,
recommendationHandler, recommendationHandler,
isFetchingFromAutosave,
toggleHERecommentation,
toggleReviewerResponse, toggleReviewerResponse,
reviewerDetailsExpanded, reviewerDetailsExpanded,
toggleEditorialComments, toggleEditorialComments,
toggleAbstractMetadata,
abstractMetadataExpanded,
toggleConflictsOfInterest,
conflictsOfInterestExpanded,
toggleFilesMetadata,
filesMetadataExpanded,
reviewerRecommendations, reviewerRecommendations,
HERecommendationExpanded,
invitationsWithReviewers, invitationsWithReviewers,
reviewerResponseExpanded, reviewerResponseExpanded,
pendingOwnRecommendation, pendingOwnRecommendation,
...@@ -105,9 +117,15 @@ const ManuscriptLayout = ({ ...@@ -105,9 +117,15 @@ const ManuscriptLayout = ({
versions={versions} versions={versions}
/> />
<ManuscriptMetadata <ManuscriptMetadata
abstractMetadataExpanded={abstractMetadataExpanded}
conflictsOfInterestExpanded={conflictsOfInterestExpanded}
currentUser={currentUser} currentUser={currentUser}
filesMetadataExpanded={filesMetadataExpanded}
fragment={fragment} fragment={fragment}
getSignedUrl={getSignedUrl} getSignedUrl={getSignedUrl}
toggleAbstractMetadata={toggleAbstractMetadata}
toggleConflictsOfInterest={toggleConflictsOfInterest}
toggleFilesMetadata={toggleFilesMetadata}
/> />
{get(currentUser, 'permissions.canViewEditorialComments', true) && ( {get(currentUser, 'permissions.canViewEditorialComments', true) && (
...@@ -138,11 +156,13 @@ const ManuscriptLayout = ({ ...@@ -138,11 +156,13 @@ const ManuscriptLayout = ({
) && ( ) && (
<ReviewerReports <ReviewerReports
currentUser={currentUser} currentUser={currentUser}
expanded={reviewReportExpanded}
getSignedUrl={getSignedUrl} getSignedUrl={getSignedUrl}
invitations={invitationsWithReviewers} invitations={invitationsWithReviewers}
isLatestVersion={isLatestVersion} isLatestVersion={isLatestVersion}
journal={journal} journal={journal}
reviewerReports={reviewerRecommendations} reviewerReports={reviewerRecommendations}
toggle={toggleReviewReport}
token={get(currentUser, 'token')} token={get(currentUser, 'token')}
/> />
)} )}
...@@ -248,24 +268,27 @@ const ManuscriptLayout = ({ ...@@ -248,24 +268,27 @@ const ManuscriptLayout = ({
canHEMakeRecommendationToPublish canHEMakeRecommendationToPublish
} }
canHEOnlyReject={canHEOnlyReject} canHEOnlyReject={canHEOnlyReject}
expanded={HERecommendationExpanded}
formValues={get(formValues, 'editorialRecommendation', {})} formValues={get(formValues, 'editorialRecommendation', {})}
highlight={reviewerRecommendations.length > 0} highlight={reviewerRecommendations.length > 0}
modalKey="heRecommendation" modalKey="heRecommendation"
onRecommendationSubmit={ onRecommendationSubmit={
recommendationHandler.onEditorialRecommendation recommendationHandler.onEditorialRecommendation
} }
toggle={toggleHERecommentation}
/> />
)} )}
{isLatestVersion && {isLatestVersion &&
get(currentUser, 'permissions.canMakeDecision', false) && ( get(currentUser, 'permissions.canMakeDecision', false) && (
<ManuscriptEicDecision <ManuscriptEicDecision
collection={collection} collection={collection}
expanded={eicDecisionExpanded}
formValues={get(formValues, 'eicDecision')} formValues={get(formValues, 'eicDecision')}
highlight={editorialRecommendations.length > 0} highlight={editorialRecommendations.length > 0}
messagesLabel={messagesLabel} messagesLabel={messagesLabel}
mt={2} mt={2}
submitDecision={recommendationHandler.createRecommendation} submitDecision={recommendationHandler.createRecommendation}
toggle={toggleEicDecision}
/> />
)} )}
</Fragment> </Fragment>
......
...@@ -293,6 +293,30 @@ export default compose( ...@@ -293,6 +293,30 @@ export default compose(
} }
}, },
}), }),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleAbstractMetadata: toggle,
abstractMetadataExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleConflictsOfInterest: toggle,
conflictsOfInterestExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleFilesMetadata: toggle,
filesMetadataExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleReviewReport: toggle,
reviewReportExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleHERecommentation: toggle,
HERecommendationExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleEicDecision: toggle,
eicDecisionExpanded: expanded,
})),
fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({ fromRenderProps(RemoteOpener, ({ toggle, expanded }) => ({
toggleAssignHE: toggle, toggleAssignHE: toggle,
heExpanded: expanded, heExpanded: expanded,
...@@ -344,8 +368,10 @@ export default compose( ...@@ -344,8 +368,10 @@ export default compose(
fetchUpdatedCollection, fetchUpdatedCollection,
editorialRecommendations, editorialRecommendations,
fragment, fragment,
collection: { status },
currentUser: { currentUser: {
isEIC, isEIC,
isReviewer,
isInvitedHE, isInvitedHE,
isInvitedToReview, isInvitedToReview,
isHEToManuscript, isHEToManuscript,
...@@ -372,6 +398,15 @@ export default compose( ...@@ -372,6 +398,15 @@ export default compose(
setEditorInChief(head(res.users)), setEditorInChief(head(res.users)),
) )
if (isEIC && status === 'pendingApproval') {
this.props.toggleEicDecision()
}
if (isReviewer && status === 'reviewCompleted')
this.props.toggleReviewReport()
if (isHEToManuscript && status === 'reviewCompleted')
this.props.toggleHERecommentation()
if (canInviteReviewers) { if (canInviteReviewers) {
getPublonsReviewers(fragmentId, setError) getPublonsReviewers(fragmentId, setError)
} }
......
...@@ -33,12 +33,14 @@ const ReviewerReports = ({ ...@@ -33,12 +33,14 @@ const ReviewerReports = ({
token, token,
invitations, invitations,
reviwerReports, reviwerReports,
...rest
}) => ( }) => (
<ContextualBox <ContextualBox
label={isLatestVersion ? 'Your Report' : 'Reviewer Reports'} label={isLatestVersion ? 'Your Report' : 'Reviewer Reports'}
mb={2} mb={2}
rightChildren={<SubmittedReports reports={reports.length} />} rightChildren={<SubmittedReports reports={reports.length} />}
startExpanded startExpanded
{...rest}
> >
{reports.map(report => ( {reports.map(report => (
<ReviewerReport <ReviewerReport
......
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