Gjøre tittel og kommentarfelt i arbeidslisten ikke obligatorisk

navikt · Nov 13, 2023 · 54b620d · 54b620d
1 parent 828f7db
commit 54b620d
Show file tree

Hide file tree

Showing 8 changed files with 56 additions and 26 deletions.
diff --git a/cypress/e2e/arbeidsliste_spec.js b/cypress/e2e/arbeidsliste_spec.js
@@ -24,13 +24,17 @@ describe('Arbeidsliste', () => {
         cy.get('.legg-i-arbeidsliste').should('not.exist');
         cy.getByTestId('legg-i-arbeidsliste_knapp').should('be.enabled').click();
         cy.get('.legg-i-arbeidsliste').should('be.visible');
+        cy.getByTestId('modal_arbeidsliste_tittel').type('valideringstest på at det ikke er lov med tegn mer enn 30');
         cy.getByTestId('modal_arbeidsliste_lagre-knapp').click();
-        cy.getByTestId('modal_arbeidsliste_form').contains('Du må fylle ut en tittel');
-        cy.getByTestId('modal_arbeidsliste_form').contains('Du må fylle ut en kommentar');
-        cy.getByTestId('modal_arbeidsliste_tittel').type('validering');
-        cy.getByTestId('modal_arbeidsliste_lagre-knapp').click();
+        cy.getByTestId('modal_arbeidsliste_form').contains('Tittelen kan ikke være lenger enn 30 tegn.');
         cy.getByTestId('modal_arbeidsliste_form').should('not.contain', 'Du må fylle ut en tittel');
-        cy.getByTestId('modal_arbeidsliste_kommentar').type('valideringskommentar');
+        cy.getByTestId('modal_arbeidsliste_kommentar').type('valideringskommentar skal ikke være lengre enn 500 tegn, så her kommer litt lorum ipsum: Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry\'s standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages.');
+        cy.getByTestId('modal_arbeidsliste_form').contains('Du må korte ned teksten til 500 tegn');
+        cy.getByTestId('modal_arbeidsliste_form').should('not.contain', 'Du må fylle ut en kommentar');
+        cy.getByTestId('modal_arbeidsliste_tittel').clear();
+        cy.getByTestId('modal_arbeidsliste_tittel').type('validering');
+        cy.getByTestId('modal_arbeidsliste_kommentar').clear();
+        cy.getByTestId('modal_arbeidsliste_kommentar').type('valideringskommentar')
         cy.get('#fristDatovelger').type('01.03.2066');
         cy.getByTestId('modal_arbeidslistekategori_GUL').click();
     });
@@ -184,19 +188,36 @@ describe('Arbeidsliste', () => {
 
         cy.getByTestId('modal_arbeidsliste_tittel').clear();
         cy.getByTestId('modal_arbeidsliste_kommentar').clear();
-        cy.getByTestId('modal_rediger-arbeidsliste_lagre-knapp').click();
-        cy.getByTestId('modal_rediger-arbeidsliste_form').contains('Du må fylle ut en tittel');
-        cy.getByTestId('modal_rediger-arbeidsliste_form').contains('Du må fylle ut en kommentar');
-
+        cy.getByTestId('modal_arbeidsliste_tittel').type('Heisann sveisann her er det mer enn 30 tegn');
+        cy.getByTestId('modal_rediger-arbeidsliste_form').contains('Tittelen kan ikke være lenger enn 30 tegn.');
+        cy.getByTestId('modal_rediger-arbeidsliste_form').should('not.contain', 'Du må fylle ut en tittel');
+        cy.getByTestId('modal_arbeidsliste_tittel').clear();
         cy.getByTestId('modal_arbeidsliste_tittel').type('Heisann sveisann');
+        cy.getByTestId('modal_arbeidsliste_kommentar').type('Her er en kjempefin kommentar truddelu');
         cy.getByTestId('modal_rediger-arbeidsliste_lagre-knapp').click();
-        cy.getByTestId('modal_rediger-arbeidsliste_form').should('not.contain', 'Du må fylle ut en tittel');
+        cy.get('.arbeidsliste-modal').should('not.exist');
+
+        cy.lukkeArbeidslistePaPerson();
+    });
+
+    it('Sjekk at man kan redigere til tom tittel og tom kommentar ', () => {
+        cy.apneArbeidslistePaPerson();
+
+        cy.get('.arbeidsliste-modal').should('not.exist');
+
+        cy.getByTestId('min-oversikt_chevron-arbeidsliste_rediger-knapp').click();
+
+        cy.get('.arbeidsliste-modal').should('be.visible');
+
+        cy.getByTestId('modal_arbeidsliste_tittel').clear();
+        cy.getByTestId('modal_arbeidsliste_kommentar').clear();
 
-        cy.getByTestId('modal_arbeidsliste_kommentar').type('Her er en kjempefin kommentar truddelu');
         cy.getByTestId('modal_rediger-arbeidsliste_lagre-knapp').click();
+
         cy.get('.arbeidsliste-modal').should('not.exist');
 
         cy.lukkeArbeidslistePaPerson();
+
     });
 
     it('Lagre tittel og kommentar', () => {

diff --git a/src/components/formik/formik-input.tsx b/src/components/formik/formik-input.tsx
@@ -14,9 +14,7 @@ function FormikInput({name, index}: FormikInputProps) {
 
     const validate = (value: string): string | undefined => {
         let error: undefined | string;
-        if (!value) {
-            error = 'Du må fylle ut en tittel';
-        } else if (value.length > TITTEL_MAKS_LENGDE) {
+        if (value.length > TITTEL_MAKS_LENGDE) {
             error = `Tittelen kan ikke være lenger enn ${TITTEL_MAKS_LENGDE} tegn.`;
         }
         return error;

diff --git a/src/components/formik/formik-tekstarea.tsx b/src/components/formik/formik-tekstarea.tsx
@@ -14,9 +14,7 @@ function FormikTekstArea({name, index}: FormikTekstAreaProps) {
 
     const validate = (value: string) => {
         let error: undefined | string;
-        if (!value) {
-            error = 'Du må fylle ut en kommentar';
-        } else if (value.length > KOMMENTAR_MAKS_LENGDE) {
+        if (value.length > KOMMENTAR_MAKS_LENGDE) {
             error = `Du må korte ned teksten til ${KOMMENTAR_MAKS_LENGDE} tegn`;
         }
         return error;

diff --git a/src/components/modal/arbeidsliste/arbeidsliste-modal-rediger.tsx b/src/components/modal/arbeidsliste/arbeidsliste-modal-rediger.tsx
@@ -35,7 +35,13 @@ interface FormikPropsValues {
     kategori: KategoriModell;
 }
 
-function ArbeidslisteModalRediger({bruker, sistEndretAv, sistEndretDato, settMarkert}: ArbeidslisteModalRedigerProps) {
+function ArbeidslisteModalRediger({
+    bruker,
+    sistEndretAv,
+    sistEndretDato,
+    settMarkert,
+    innloggetVeileder
+}: ArbeidslisteModalRedigerProps) {
     const arbeidslisteStatus = useSelector((state: AppState) => state.arbeidsliste.status);
     const statusLaster = arbeidslisteStatus !== undefined && arbeidslisteStatus === STATUS.PENDING;
     const [isOpen, setIsOpen] = useState(false);
@@ -95,7 +101,13 @@ function ArbeidslisteModalRediger({bruker, sistEndretAv, sistEndretDato, settMar
                     onSubmit={values => {
                         setIsOpen(false);
                         dispatch(
-                            redigerArbeidslisteAction(values, {bruker, sistEndretAv, sistEndretDato, settMarkert})
+                            redigerArbeidslisteAction(values, {
+                                bruker,
+                                sistEndretAv,
+                                sistEndretDato,
+                                settMarkert,
+                                innloggetVeileder
+                            })
                         );
                         trackAmplitude(
                             {

diff --git a/src/components/modal/arbeidsliste/legg-til-arbeidslisteform.tsx b/src/components/modal/arbeidsliste/legg-til-arbeidslisteform.tsx
@@ -193,8 +193,8 @@ const mapDispatchToProps = (dispatch, props) => ({
         const {valgteBrukere} = props;
         const liste = arbeidsliste.map((elem, index) => ({
             fnr: valgteBrukere[index].fnr,
-            overskrift: elem.overskrift,
-            kommentar: elem.kommentar,
+            overskrift: elem.overskrift.length ? elem.overskrift : null,
+            kommentar: elem.kommentar.length ? elem.kommentar : null,
             frist: elem.frist ? dateToISODate(elem.frist) : null,
             kategori: elem.kategori
         }));

diff --git a/src/ducks/arbeidsliste.ts b/src/ducks/arbeidsliste.ts
@@ -50,8 +50,8 @@ export default function arbeidslisteReducer(state = initialState, action) {
 // Action Creators
 export function redigerArbeidslisteAction(formData, props) {
     const arbeidsliste = {
-        kommentar: formData.kommentar,
-        overskrift: formData.overskrift,
+        kommentar: formData.kommentar.length ? formData.kommentar : null,
+        overskrift: formData.overskrift.length ? formData.overskrift : null,
         frist: formData.frist ? dateToISODate(formData.frist) : null,
         kategori: formData.kategori
     };

diff --git a/src/minoversikt/minoversikt-arbeidslistepanel.tsx b/src/minoversikt/minoversikt-arbeidslistepanel.tsx
@@ -25,7 +25,8 @@ export default function ArbeidslistePanel({
         bruker.arbeidsliste.sistEndretAv && bruker.arbeidsliste.sistEndretAv.veilederId
             ? bruker.arbeidsliste.sistEndretAv.veilederId
             : String.fromCharCode(8212);
-    const overskrift = !!bruker.arbeidsliste.overskrift ? bruker.arbeidsliste.overskrift : String.fromCharCode(8212);
+    const overskrift = !!bruker.arbeidsliste.overskrift ? bruker.arbeidsliste.overskrift : 'Ingen tekst i tittel';
+    const kommentar = !!bruker.arbeidsliste.kommentar ? bruker.arbeidsliste.kommentar : 'Ingen tekst i kommentar';
 
     let arbeidslisteFristTekst;
     if (bruker.arbeidsliste.frist != null) {
@@ -44,7 +45,7 @@ export default function ArbeidslistePanel({
                         Arbeidsliste frist: {arbeidslisteFristTekst}
                     </Detail>
                     <BodyShort size="small" data-testid="chevron_arbeidslisteinnhold_kommentar">
-                        {bruker.arbeidsliste.kommentar}
+                        {kommentar}
                     </BodyShort>
                     {!bruker.arbeidsliste.hentetKommentarOgTittel && (
                         <Loader variant="neutral" size="xsmall" title="Henter arbeidsliste for bruker..." />

diff --git a/src/model-interfaces.ts b/src/model-interfaces.ts
@@ -263,7 +263,7 @@ export enum Status {
 
 export interface ArbeidslisteDataModell {
     fnr: string;
-    kommentar: string;
+    kommentar: string | null;
     frist: string | null;
     kategori: KategoriModell | null;
 }