From 22f98105e9184ae4e25f58cb3c57983ee5fd908a Mon Sep 17 00:00:00 2001 From: albina Date: Mon, 21 Oct 2024 10:35:25 +0000 Subject: [PATCH 1/9] tweaks --- .../Constituency/index.tsx | 133 +++++++++--------- .../src/screens-parliamentary/List/index.tsx | 14 +- .../src/screens-parliamentary/index.tsx | 12 +- .../extendDeadline/index.tsx | 18 +-- 4 files changed, 96 insertions(+), 81 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx index 0f5fd355a24b..c9b9ca3f7062 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx @@ -127,7 +127,7 @@ export const Constituency = ({ ': ' + constituencyLists.length} - {constituencyLists?.length > 0 && ( + {allowedToProcess && constituencyLists?.length > 0 && ( )} @@ -156,77 +156,80 @@ export const Constituency = ({ }} tag={ !list.reviewed - ? { - label: 'Cancel collection', - renderTag: () => ( - - - - - - } - onConfirm={() => { - removeList({ - variables: { - input: { - listId: list.id, - }, - }, - }) - - if ( + ? allowedToProcess + ? { + label: 'Cancel collection', + renderTag: () => ( + + + + + + } + onConfirm={() => { + removeList({ variables: { input: { - candidateId: list.candidate.id, + listId: list.id, }, }, }) + + if ( + candidatesListCount[list.candidate.id] === + 1 + ) { + removeCandidate({ + variables: { + input: { + candidateId: list.candidate.id, + }, + }, + }) + } + }} + buttonTextConfirm={ + candidatesListCount[list.candidate.id] === 1 + ? formatMessage( + m.cancelCollectionAndCandidateModalConfirmButton, + ) + : formatMessage( + m.cancelCollectionModalConfirmButton, + ) } - }} - buttonTextConfirm={ - candidatesListCount[list.candidate.id] === 1 - ? formatMessage( - m.cancelCollectionAndCandidateModalConfirmButton, - ) - : formatMessage( - m.cancelCollectionModalConfirmButton, - ) - } - buttonPropsConfirm={{ - variant: 'primary', - colorScheme: 'destructive', - }} - buttonTextCancel={formatMessage( - m.cancelCollectionModalCancelButton, - )} - /> - ), - } + buttonPropsConfirm={{ + variant: 'primary', + colorScheme: 'destructive', + }} + buttonTextCancel={formatMessage( + m.cancelCollectionModalCancelButton, + )} + /> + ), + } + : undefined : { label: m.confirmListReviewed.defaultMessage, variant: 'mint', diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx index b342542ddc29..adb2eeb15c88 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx @@ -91,10 +91,18 @@ const List = ({ allowedToProcess }: { allowedToProcess: boolean }) => { } type={listStatus === ListStatus.Reviewed ? 'success' : undefined} /> - + - - + {allowedToProcess && ( + + + + + )} diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/index.tsx index 3a0b0b940abf..cc6f244a3984 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/index.tsx @@ -101,10 +101,12 @@ const ParliamentaryRoot = ({ backgroundColor="blue" /> - + {allowedToProcess && ( + + )} {loading && ( @@ -202,7 +204,7 @@ const ParliamentaryRoot = ({ /> ))} - + {allowedToProcess && } diff --git a/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx b/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx index c37ee13b4141..871f82f9f927 100644 --- a/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx +++ b/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx @@ -63,14 +63,16 @@ const ActionExtendDeadline = ({ readOnly value={format(new Date(endDate), 'dd.MM.yyyy HH:mm')} /> - - - + {allowedToProcess && ( + + + + )} Date: Mon, 21 Oct 2024 10:43:05 +0000 Subject: [PATCH 2/9] lks - hide signees while active --- .../Constituency/index.tsx | 28 ++++++++----------- .../src/screens-parliamentary/List/index.tsx | 8 ++++-- .../completeReview/index.tsx | 2 +- 3 files changed, 19 insertions(+), 19 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx index ba8b75cfa3df..c9b9ca3f7062 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx @@ -142,22 +142,18 @@ export const Constituency = ({ maxProgress: list.area.min, withLabel: true, }} - cta={ - (allowedToProcess && !list.active) || !allowedToProcess - ? { - label: formatMessage(m.viewList), - variant: 'text', - onClick: () => { - navigate( - SignatureCollectionPaths.ParliamentaryConstituencyList.replace( - ':constituencyName', - constituencyName, - ).replace(':listId', list.id), - ) - }, - } - : undefined - } + cta={{ + label: formatMessage(m.viewList), + variant: 'text', + onClick: () => { + navigate( + SignatureCollectionPaths.ParliamentaryConstituencyList.replace( + ':constituencyName', + constituencyName, + ).replace(':listId', list.id), + ) + }, + }} tag={ !list.reviewed ? allowedToProcess diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx index adb2eeb15c88..85b6c238d40d 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx @@ -96,10 +96,14 @@ const List = ({ allowedToProcess }: { allowedToProcess: boolean }) => { endTime={list.endTime} allowedToProcess={allowedToProcess} /> - + {(allowedToProcess && !list.active) || !allowedToProcess && + + } {allowedToProcess && ( - + {!list.active && + + } )} diff --git a/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx b/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx index 96b5b1488381..28c9e1723236 100644 --- a/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx +++ b/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx @@ -41,7 +41,7 @@ const ActionReviewComplete = ({ }) return ( - + diff --git a/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx b/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx index 871f82f9f927..b49d827c5f7e 100644 --- a/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx +++ b/libs/portals/admin/signature-collection/src/shared-components/extendDeadline/index.tsx @@ -63,16 +63,15 @@ const ActionExtendDeadline = ({ readOnly value={format(new Date(endDate), 'dd.MM.yyyy HH:mm')} /> - {allowedToProcess && ( - - - - )} + + + Date: Mon, 21 Oct 2024 11:07:23 +0000 Subject: [PATCH 5/9] action disabled --- .../src/shared-components/completeReview/index.tsx | 4 ++-- .../src/shared-components/completeReview/lockList/index.tsx | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx b/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx index b931adb3e942..5240b0db9dfc 100644 --- a/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx +++ b/libs/portals/admin/signature-collection/src/shared-components/completeReview/index.tsx @@ -6,7 +6,7 @@ import { useToggleListReviewMutation } from './toggleListReview.generated' import { useRevalidator } from 'react-router-dom' import { m } from '../../lib/messages' import { ListStatus } from '../../lib/utils' -import LockList from './lockList' +import ActionLockList from './lockList' const ActionReviewComplete = ({ listId, @@ -43,7 +43,7 @@ const ActionReviewComplete = ({ return ( - + From b65ad5da5450cfb752676112725a7f572fba62d9 Mon Sep 17 00:00:00 2001 From: albina Date: Mon, 21 Oct 2024 11:31:41 +0000 Subject: [PATCH 6/9] messages cleanup --- .../signature-collection/src/lib/messages.ts | 52 ++++++------------- .../src/lib/navigation.ts | 4 +- .../Constituency/index.tsx | 8 +-- .../src/screens-parliamentary/index.tsx | 8 +-- .../screens-presidential/AllLists/index.tsx | 2 +- .../createCollection/index.tsx | 3 +- 6 files changed, 29 insertions(+), 48 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/lib/messages.ts b/libs/portals/admin/signature-collection/src/lib/messages.ts index 624895b9f682..704860b4443f 100644 --- a/libs/portals/admin/signature-collection/src/lib/messages.ts +++ b/libs/portals/admin/signature-collection/src/lib/messages.ts @@ -13,7 +13,7 @@ export const m = defineMessages({ description: '', }, signatureListsTitlePresidential: { - id: 'admin-portal.signature-collection-parliamentary:signatureLists', + id: 'admin-portal.signature-collection:signatureListsTitlePresidential', defaultMessage: 'Forsetakosningar', description: '', }, @@ -166,6 +166,22 @@ export const m = defineMessages({ defaultMessage: 'Forsetakosningar 2024', description: '', }, + parliamentaryCollectionTitle: { + id: 'admin-portal.signature-collection:parliamentaryCollectionTitle', + defaultMessage: 'Alþingiskosningar', + description: '', + }, + parliamentaryCollectionIntro: { + id: 'admin-portal.signature-collection:parliamentaryCollectionIntro', + defaultMessage: 'Hér er yfirlit yfir kjördæmin sex. Ýttu á viðeigandi kjördæmi til að sjá meðmælendalista í því kjördæmi.', + description: '', + }, + parliamentaryConstituencyIntro: { + id: 'admin-portal.signature-collection:parliamentaryConstituencyIntro', + defaultMessage: + 'Hér er yfirlit yfir allar meðmælasafnanir sem stofnaðar hafa verið í', + description: '', + }, viewList: { id: 'admin-portal.signature-collection:viewList', defaultMessage: 'Skoða söfnun', @@ -713,40 +729,6 @@ export const m = defineMessages({ }, }) -export const parliamentaryMessages = defineMessages({ - signatureListsTitle: { - id: 'admin-portal.signature-collection-parliamentary:signatureLists', - defaultMessage: 'Alþingiskosningar', - description: '', - }, - signatureListsDescription: { - id: 'admin-portal.signature-collection-parliamentary:signatureListsDescription', - defaultMessage: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit.', - description: '', - }, - signatureListsIntro: { - id: 'admin-portal.signature-collection-parliamentary:signatureListsIntro', - defaultMessage: - 'Hér er yfirlit yfir kjördæmin sex. Ýttu á viðeigandi kjördæmi til að sjá meðmælendalista í því kjördæmi.', - description: '', - }, - compareListsButton: { - id: 'admin-portal.signature-collection-parliamentary:compareListsButton', - defaultMessage: 'Bera saman', - description: '', - }, - compareListsDescription: { - id: 'admin-portal.signature-collection-parliamentary:compareListsDescription', - defaultMessage: 'Lorem ipsum dolor sit amet, consectetur adipiscing elit', - description: '', - }, - singleConstituencyIntro: { - id: 'admin-portal.signature-collection-parliamentary:singleConstituencyIntro', - defaultMessage: - 'Hér er yfirlit yfir allar meðmælasafnanir sem stofnaðar hafa verið í', - description: '', - }, -}) export const createCollectionErrorMessages = defineMessages({ age: { diff --git a/libs/portals/admin/signature-collection/src/lib/navigation.ts b/libs/portals/admin/signature-collection/src/lib/navigation.ts index c48d371f7453..419e13dc59d7 100644 --- a/libs/portals/admin/signature-collection/src/lib/navigation.ts +++ b/libs/portals/admin/signature-collection/src/lib/navigation.ts @@ -1,6 +1,6 @@ import { PortalNavigationItem } from '@island.is/portals/core' import { SignatureCollectionPaths } from './paths' -import { m, parliamentaryMessages } from './messages' +import { m } from './messages' export const signatureCollectionNavigation: PortalNavigationItem = { name: m.signatureListsTitle, @@ -11,7 +11,7 @@ export const signatureCollectionNavigation: PortalNavigationItem = { path: SignatureCollectionPaths.ParliamentaryRoot, children: [ { - name: parliamentaryMessages.signatureListsTitle, + name: m.parliamentaryCollectionTitle, path: SignatureCollectionPaths.ParliamentaryRoot, activeIfExact: true, }, diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx index c9b9ca3f7062..6e2f196eb529 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx @@ -1,7 +1,7 @@ import { IntroHeader, PortalNavigation } from '@island.is/portals/core' import { signatureCollectionNavigation } from '../../lib/navigation' import { useLocale } from '@island.is/localization' -import { m, parliamentaryMessages } from '../../lib/messages' +import { m } from '../../lib/messages' import { ActionCard, Box, @@ -91,7 +91,7 @@ export const Constituency = ({ items={[ { title: formatMessage( - parliamentaryMessages.signatureListsTitle, + m.parliamentaryCollectionTitle, ), href: `/stjornbord${SignatureCollectionPaths.ParliamentaryRoot}`, }, @@ -104,7 +104,7 @@ export const Constituency = ({ { tag={ list.reviewed ? { - label: m.confirmListReviewed.defaultMessage, + label: formatMessage(m.confirmListReviewed), variant: 'mint', outlined: false, } diff --git a/libs/portals/admin/signature-collection/src/shared-components/createCollection/index.tsx b/libs/portals/admin/signature-collection/src/shared-components/createCollection/index.tsx index 73cb544d1093..12c03e5c9596 100644 --- a/libs/portals/admin/signature-collection/src/shared-components/createCollection/index.tsx +++ b/libs/portals/admin/signature-collection/src/shared-components/createCollection/index.tsx @@ -153,7 +153,6 @@ const CreateCollection = ({ collectionId }: { collectionId: string }) => { label={formatMessage(m.candidateName)} readOnly value={name} - placeholder={loadingCandidate ? 'Sæki nafn...' : ''} /> {!canCreate && ( @@ -170,7 +169,7 @@ const CreateCollection = ({ collectionId }: { collectionId: string }) => { onClick={() => { createNewCollection() }} - disabled={!canCreate} + disabled={!canCreate || !name} loading={loading} > {formatMessage(m.createCollection)} From e560fbd91a3bce99c82deee4f091e0b8e1a735bc Mon Sep 17 00:00:00 2001 From: albina Date: Mon, 21 Oct 2024 11:36:52 +0000 Subject: [PATCH 7/9] fixes --- libs/portals/admin/signature-collection/src/lib/messages.ts | 4 ++-- .../src/screens-parliamentary/Constituency/index.tsx | 4 +--- .../signature-collection/src/screens-parliamentary/index.tsx | 4 +--- 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/lib/messages.ts b/libs/portals/admin/signature-collection/src/lib/messages.ts index 704860b4443f..f5ba0db59386 100644 --- a/libs/portals/admin/signature-collection/src/lib/messages.ts +++ b/libs/portals/admin/signature-collection/src/lib/messages.ts @@ -173,7 +173,8 @@ export const m = defineMessages({ }, parliamentaryCollectionIntro: { id: 'admin-portal.signature-collection:parliamentaryCollectionIntro', - defaultMessage: 'Hér er yfirlit yfir kjördæmin sex. Ýttu á viðeigandi kjördæmi til að sjá meðmælendalista í því kjördæmi.', + defaultMessage: + 'Hér er yfirlit yfir kjördæmin sex. Ýttu á viðeigandi kjördæmi til að sjá meðmælendalista í því kjördæmi.', description: '', }, parliamentaryConstituencyIntro: { @@ -729,7 +730,6 @@ export const m = defineMessages({ }, }) - export const createCollectionErrorMessages = defineMessages({ age: { id: 'admin-portal.signature-collection:error.age', diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx index 6e2f196eb529..cc7ae4cd8152 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/Constituency/index.tsx @@ -90,9 +90,7 @@ export const Constituency = ({ From 3177163c1309920ca5b84def22ce5d49dad6f3d4 Mon Sep 17 00:00:00 2001 From: albina Date: Mon, 21 Oct 2024 12:00:47 +0000 Subject: [PATCH 8/9] build fix --- .../src/screens-parliamentary/List/index.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx index 7366efc15e9f..08e322264228 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx @@ -8,7 +8,7 @@ import { import { useLocale } from '@island.is/localization' import { IntroHeader, PortalNavigation } from '@island.is/portals/core' import { signatureCollectionNavigation } from '../../lib/navigation' -import { m, parliamentaryMessages } from '../../lib/messages' +import { m } from '../../lib/messages' import { useLoaderData } from 'react-router-dom' import { ListStatus, SignatureCollectionList } from '@island.is/api/schema' import { PaperSignees } from './paperSignees' @@ -49,7 +49,7 @@ const List = ({ allowedToProcess }: { allowedToProcess: boolean }) => { items={[ { title: formatMessage( - parliamentaryMessages.signatureListsTitle, + m.parliamentaryCollectionTitle, ), href: `/stjornbord${SignatureCollectionPaths.ParliamentaryRoot}`, }, From 5aa69672b25b4c42998f9cf2dcb2422543ea0dbe Mon Sep 17 00:00:00 2001 From: andes-it Date: Mon, 21 Oct 2024 12:14:50 +0000 Subject: [PATCH 9/9] chore: nx format:write update dirty files --- .../src/screens-parliamentary/List/index.tsx | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx index 08e322264228..48c8d6d1969e 100644 --- a/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx +++ b/libs/portals/admin/signature-collection/src/screens-parliamentary/List/index.tsx @@ -48,9 +48,7 @@ const List = ({ allowedToProcess }: { allowedToProcess: boolean }) => {