Feature/ep approval permissions#824
Merged
zzacharo merged 3 commits intoJun 18, 2026
Merged
Annotations
10 errors and 3 warnings
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L124
Replace `·userAvatar={userAvatar}·request={request}·permissions={permissions}` with `⏎··········userAvatar={userAvatar}⏎··········request={request}⏎··········permissions={permissions}⏎·······`
|
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L77
Replace `·—·{i18next.t("{{n}}·non-signer(s)",·{·n:·payload.num_non_signers·})}` with `⏎················{"·"}⏎················—{"·"}⏎················{i18next.t("{{n}}·non-signer(s)",·{⏎··················n:·payload.num_non_signers,⏎················})}⏎··············`
|
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L73
Replace `{i18next.t("Paper·signed·by·whole·collaboration")}` with `⏎············{i18next.t("Paper·signed·by·whole·collaboration")}⏎··········`
|
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L51
Replace `·href={payload.latest_version_url}·target="_blank"·rel="noreferrer"` with `⏎················href={payload.latest_version_url}⏎················target="_blank"⏎················rel="noreferrer"⏎··············`
|
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L14
Replace `<>{value·?·i18next.t("Yes")·:·i18next.t("No")}</>` with `(⏎··<>{value·?·i18next.t("Yes")·:·i18next.t("No")}</>⏎)`
|
|
Run eslint test:
assets/js/components/record_details/EPApproval.js#L303
Replace `⏎··························href={requestLink}⏎··························target="_blank"⏎··························rel="noreferrer"⏎························` with `·href={requestLink}·target="_blank"·rel="noreferrer"`
|
|
Run eslint test:
assets/js/components/record_details/CreatePublicRecordModal.js#L205
Replace `·floated="left"·onClick={this.handleClose}·disabled={submitting}` with `⏎············floated="left"⏎············onClick={this.handleClose}⏎············disabled={submitting}⏎··········`
|
|
Run eslint test:
assets/js/components/record_details/CreatePublicRecordModal.js#L143
Replace `·this.setState({·agreedToTerms:·checked·})` with `⏎··················this.setState({·agreedToTerms:·checked·})⏎················`
|
|
Run eslint test:
assets/js/components/record_details/CreatePublicRecordModal.js#L116
Replace `{i18next.t("Public·record·created")}` with `⏎················{i18next.t("Public·record·created")}⏎··············`
|
|
Run eslint test:
assets/js/components/record_details/CreatePublicRecordModal.js#L9
Replace `·Button,·Checkbox,·Header,·Icon,·Message,·Modal·` with `⏎··Button,⏎··Checkbox,⏎··Header,⏎··Icon,⏎··Message,⏎··Modal,⏎`
|
|
Run eslint test:
assets/js/components/requests/overrides/TimelineEventBody.js#L7
'React' is defined but never used
|
|
Run eslint test:
assets/js/components/requests/overrides/LockRequest.js#L7
'React' is defined but never used
|
|
Run eslint test:
assets/js/components/requests/EPApprovalRequestDetails.js#L14
Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all
|
background
wait
wait-all
cancel
parallel
Loading