Skip to content

Commit

Permalink
fix: small typo fixes for FAP and Proposal templates (#743)
Browse files Browse the repository at this point in the history
Co-authored-by: janosbabik <143906591+janosbabik@users.noreply.github.com>
  • Loading branch information
bolmsten and janosbabik authored Sep 12, 2024
1 parent 7bd13fb commit ffe4bf8
Show file tree
Hide file tree
Showing 14 changed files with 25 additions and 25 deletions.
2 changes: 1 addition & 1 deletion apps/e2e/cypress/e2e/FAPs.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,7 @@ context('Fap reviews tests', () => {

cy.contains(proposal1.title).parent().find('[type="checkbox"]').check();

cy.get("[aria-label='Assign proposals to Fap']").first().click();
cy.get("[aria-label='Assign proposals to FAP']").first().click();

cy.get('[data-cy="fap-selection"] input').should(
'not.have.class',
Expand Down
4 changes: 2 additions & 2 deletions apps/e2e/cypress/e2e/proposalAdministration.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ context('Proposal administration tests', () => {
cy.get('[data-cy="change-proposal-status"]').click();

cy.get('[role="presentation"] .MuiDialogContent-root').as('dialog');
cy.get('@dialog').contains('Change proposal/s status');
cy.get('@dialog').contains('Change proposal(s) status');

cy.get('@dialog')
.find('#selectedStatusId-input')
Expand Down Expand Up @@ -271,7 +271,7 @@ context('Proposal administration tests', () => {
cy.get('[data-cy="change-proposal-status"]').click();

cy.get('[role="presentation"] .MuiDialogContent-root').as('dialog');
cy.get('@dialog').contains('Change proposal/s status');
cy.get('@dialog').contains('Change proposal(s) status');

cy.get('@dialog')
.find('#selectedStatusId-input')
Expand Down
12 changes: 6 additions & 6 deletions apps/e2e/cypress/e2e/proposals.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,7 @@ context('Proposal tests', () => {
.find('input')
.uncheck();
cy.get('.MuiPopover-paper')
.contains('Fap')
.contains('FAP')
.parent()
.find('input')
.uncheck();
Expand All @@ -366,7 +366,7 @@ context('Proposal tests', () => {
);
cy.get('[data-cy="officer-proposals-table"] table').should(
'not.contain',
'Fap'
'FAP'
);

cy.get("[aria-label='Show Columns']").first().click();
Expand All @@ -376,7 +376,7 @@ context('Proposal tests', () => {
.find('input')
.check();
cy.get('.MuiPopover-paper')
.contains('Fap')
.contains('FAP')
.parent()
.find('input')
.check();
Expand All @@ -389,7 +389,7 @@ context('Proposal tests', () => {
);
cy.get('[data-cy="officer-proposals-table"] table').should(
'contain',
'Fap'
'FAP'
);
});

Expand Down Expand Up @@ -620,7 +620,7 @@ context('Proposal tests', () => {
cy.get('[data-cy="change-proposal-status"]').click();

cy.get('[role="presentation"] .MuiDialogContent-root').as('dialog');
cy.get('@dialog').contains('Change proposal/s status');
cy.get('@dialog').contains('Change proposal(s) status');

cy.get('@dialog')
.find('#selectedStatusId-input')
Expand All @@ -644,7 +644,7 @@ context('Proposal tests', () => {
cy.get('[data-cy="change-proposal-status"]').click();

cy.get('[role="presentation"] .MuiDialogContent-root').as('dialog');
cy.get('@dialog').contains('Change proposal/s status');
cy.get('@dialog').contains('Change proposal(s) status');

cy.get('@dialog')
.find('#selectedStatusId-input')
Expand Down
2 changes: 1 addition & 1 deletion apps/e2e/cypress/e2e/settings.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -959,7 +959,7 @@ context('Settings tests', () => {

cy.get('[type="checkbox"]').first().check();

cy.get("[aria-label='Assign proposals to Fap']").first().click();
cy.get("[aria-label='Assign proposals to FAP']").first().click();

cy.get('[data-cy="fap-selection"] input').should(
'not.have.class',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ const AssignInstrumentsToCall = ({
{ title: 'Short code', field: 'shortCode' },
{ title: 'Description', field: 'description' },
{
title: 'Fap',
title: 'FAP',
field: 'fap',
render: (rowData: Instrument) => (
<FapSelectionComponent {...rowData} onChange={onChange} />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ const AssignedInstrumentsTable = ({
editable: 'never',
},
{
title: 'Fap',
title: 'FAP',
field: 'fapId',
editable: 'onUpdate',
emptyValue: '-',
Expand Down
4 changes: 2 additions & 2 deletions apps/frontend/src/components/call/CallGeneralInfo.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,7 @@ const CallGeneralInfo = ({
<FormControl fullWidth>
<FormikUIAutocomplete
name="templateId"
label="Call template"
label="Proposal template"
loading={loadingTemplates}
noOptionsText="No templates"
items={templateOptions}
Expand Down Expand Up @@ -364,7 +364,7 @@ const CallGeneralInfo = ({
<FormControl fullWidth>
<FormikUIAutocomplete
name="esiTemplateId"
label="ESI template"
label="Proposal ESI template"
loading={loadingTemplates}
noOptionsText="No templates"
items={esiTemplateOptions}
Expand Down
2 changes: 1 addition & 1 deletion apps/frontend/src/components/call/CallReviewsInfo.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ const CallReviewAndNotification = () => {
</LocalizationProvider>
<FormikUIAutocomplete
name="faps"
label="Call Faps"
label="Call FAPs"
multiple
loading={loadingFaps}
noOptionsText="No Faps"
Expand Down
8 changes: 4 additions & 4 deletions apps/frontend/src/components/call/CallsTable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -117,20 +117,20 @@ const CallsTable = ({ confirm }: WithConfirmProps) => {
emptyValue: '-',
},
{
title: 'Call template',
title: 'Proposal template',
field: 'template.name',
emptyValue: '-',
},
{
title: '#' + i18n.format(t('instrument'), 'plural'),
title: '#' + i18n.format(t('Instrument'), 'plural'),
render: (data) => data.instruments.length,
},
{
title: '#proposals',
title: '#Proposals',
field: 'proposalCount',
},
{
title: '#faps',
title: '#FAPs',
render: (data) => data.faps?.length,
},
];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ const FapMeetingInstrumentsTable = ({
},
{
title: 'Submit ' + i18n.format(t('instrument'), 'lowercase'),
description: `No further changes to Fap meeting decisions and rankings are possible after submission. Are you sure you want to submit the ${(rowData as InstrumentWithAvailabilityTime).name} ${t(
description: `No further changes to FAP meeting decisions and rankings are possible after submission. Are you sure you want to submit the ${(rowData as InstrumentWithAvailabilityTime).name} ${t(
'instrument'
)}?`,
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ const OverwriteTimeAllocationDialog = ({
id="fapTimeAllocation"
type="number"
name="fapTimeAllocation"
label={`Time Allocation(${fapProposalArgs.proposal.call?.allocationTimeUnit}s)`}
label={`Time Allocation (${fapProposalArgs.proposal.call?.allocationTimeUnit}s)`}
value={values.fapTimeAllocation ?? ''}
onChange={(e: React.ChangeEvent<HTMLInputElement>) => {
setFieldValue(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ const ChangeProposalStatus = ({
padding: '22px 0 0',
}}
>
Change proposal/s status
Change proposal(s) status
</Typography>

<Grid container spacing={3}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ const instrumentManagementColumns = (

const FapReviewColumns = [
{ title: 'Final status', field: 'finalStatus', emptyValue: '-' },
{ title: 'Fap', field: 'fapCode', emptyValue: '-', hidden: true },
{ title: 'FAP', field: 'fapCode', emptyValue: '-', hidden: true },
];

const proposalStatusFilter: Record<string, number> = {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ const FapReviewColumns = [
hidden: true,
},
{
title: 'Fap',
title: 'FAP',
field: 'faps.code',
render: (rowData: ProposalViewData) =>
fromArrayToCommaSeparated(rowData.faps?.map((fap) => fap.code)),
Expand Down Expand Up @@ -981,7 +981,7 @@ const ProposalTableOfficer = ({
},
{
icon: GroupWorkIcon,
tooltip: 'Assign proposals to Fap',
tooltip: 'Assign proposals to FAP',
onClick: () => {
setOpenAssignment(true);
},
Expand Down

0 comments on commit ffe4bf8

Please sign in to comment.