IS-1488: Removed unused toggles + added new one

navikt · Aug 29, 2023 · ad2fe86 · ad2fe86
1 parent deea096
commit ad2fe86
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 91 deletions.
diff --git a/server/unleash.ts b/server/unleash.ts
@@ -11,7 +11,7 @@ export type Toggles = {
 // See toggles: https://teamsykefravr-unleash-web.nav.cloud.nais.io/features
 export enum ToggleNames {
   isVirksomhetsinputEnabled = "isVirksomhetsinputEnabled",
-  isReturLegeerklaringEnabled = "isReturLegeerklaringEnabled",
+  isMeldingsTypeMeldingFraNAVEnabled = "isMeldingsTypeMeldingFraNAVEnabled",
   isMotebehovTilbakemeldingEnabled = "isMotebehovTilbakemeldingEnabled",
 }
 
@@ -52,12 +52,8 @@ export const getToggles = (veilederId, enhetId) => {
       "isVirksomhetsinputEnabled",
       context
     ),
-    isBehandlerDialogLegeerklaringEnabled: unleash.isEnabled(
-      "isBehandlerDialogLegeerklaringEnabled",
-      context
-    ),
-    isReturLegeerklaringEnabled: unleash.isEnabled(
-      "isReturLegeerklaringEnabled",
+    isMeldingsTypeMeldingFraNAVEnabled: unleash.isEnabled(
+      "isMeldingsTypeMeldingFraNAVEnabled",
       context
     ),
     isMotebehovTilbakemeldingEnabled: unleash.isEnabled(

diff --git a/src/components/behandlerdialog/meldinger/MeldingerISamtale.tsx b/src/components/behandlerdialog/meldinger/MeldingerISamtale.tsx
@@ -12,7 +12,6 @@ import {
 } from "../../../../img/ImageComponents";
 import { PaminnelseMelding } from "@/components/behandlerdialog/paminnelse/PaminnelseMelding";
 import { AvvistMelding } from "@/components/behandlerdialog/meldinger/AvvistMelding";
-import { useFeatureToggles } from "@/data/unleash/unleashQueryHooks";
 import { ReturLegeerklaring } from "@/components/behandlerdialog/legeerklaring/ReturLegeerklaring";
 
 const StyledWrapper = styled.div`
@@ -59,18 +58,14 @@ const MeldingFraBehandler = ({
   meldinger,
   melding,
 }: MeldingFraBehandlerProps) => {
-  const { toggles } = useFeatureToggles();
   const isLegeerklaring =
     melding.type === MeldingType.FORESPORSEL_PASIENT_LEGEERKLARING;
   const sentReturForLegeerklaring = meldinger.some(
     (m) =>
       m.type === MeldingType.HENVENDELSE_RETUR_LEGEERKLARING &&
       m.parentRef === melding.uuid
   );
-  const showReturLegeerklaring =
-    toggles.isReturLegeerklaringEnabled &&
-    isLegeerklaring &&
-    !sentReturForLegeerklaring;
+  const showReturLegeerklaring = isLegeerklaring && !sentReturForLegeerklaring;
 
   return (
     <StyledMelding innkommende>

diff --git a/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandler.tsx b/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandler.tsx
@@ -2,14 +2,13 @@ import React from "react";
 import { MeldingTilBehandlerSkjema } from "@/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandlerSkjema";
 import { Alert, Heading } from "@navikt/ds-react";
 import styled from "styled-components";
-import { useFeatureToggles } from "@/data/unleash/unleashQueryHooks";
-import AppSpinner from "@/components/AppSpinner";
 
 const texts = {
   header: "Skriv til behandler",
   tilleggsopplysningerInfo:
     "Her kan du kun be om tilleggsopplysninger med takst L8. Dialogmeldingen skal bare benyttes i sykefraværsoppfølgingen. Meldingen vises også til innbyggeren på Min side.",
-  meldingVisesTilBruker: "Meldingen vises til innbyggeren på Min side.",
+  alertInfo:
+    "Dialogmeldingen skal bare benyttes i sykefraværsoppfølgingen. Meldingen vises til innbyggeren på Min side.",
 };
 
 const MeldingTilBehandlerAlert = styled(Alert)`
@@ -22,29 +21,15 @@ const MeldingTilBehandlerAlert = styled(Alert)`
 `;
 
 export const MeldingTilBehandler = () => {
-  const { toggles, isLoading } = useFeatureToggles();
-
   return (
     <>
       <Heading level="1" size="large" spacing>
         {texts.header}
       </Heading>
-      {isLoading ? (
-        <AppSpinner />
-      ) : (
-        <>
-          <MeldingTilBehandlerAlert variant="warning" size="small">
-            {toggles.isBehandlerDialogLegeerklaringEnabled
-              ? texts.meldingVisesTilBruker
-              : texts.tilleggsopplysningerInfo}
-          </MeldingTilBehandlerAlert>
-          <MeldingTilBehandlerSkjema
-            isBehandlerdialogLegeerklaringEnabled={
-              toggles.isBehandlerDialogLegeerklaringEnabled
-            }
-          />
-        </>
-      )}
+      <MeldingTilBehandlerAlert variant="warning" size="small">
+        {texts.alertInfo}
+      </MeldingTilBehandlerAlert>
+      <MeldingTilBehandlerSkjema />
     </>
   );
 };
diff --git a/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandlerSkjema.tsx b/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandlerSkjema.tsx
@@ -74,13 +74,7 @@ type MeldingTilBehandlerSkjemaFeil = {
 
 export const MAX_LENGTH_BEHANDLER_MELDING = 2000; // TODO: må bli enige om noe her
 
-interface Props {
-  isBehandlerdialogLegeerklaringEnabled: boolean;
-}
-
-export const MeldingTilBehandlerSkjema = ({
-  isBehandlerdialogLegeerklaringEnabled,
-}: Props) => {
+export const MeldingTilBehandlerSkjema = () => {
   const [displayPreview, setDisplayPreview] = useState(false);
   const { getMeldingTilBehandlerDocument } = useMeldingTilBehandlerDocument();
   const [selectedBehandler, setSelectedBehandler] = useState<BehandlerDTO>();
@@ -128,28 +122,18 @@ export const MeldingTilBehandlerSkjema = ({
   };
 
   return (
-    <Form
-      onSubmit={submit}
-      validate={validate}
-      initialValues={{
-        type: isBehandlerdialogLegeerklaringEnabled
-          ? undefined
-          : MeldingType.FORESPORSEL_PASIENT_TILLEGGSOPPLYSNINGER,
-      }}
-    >
+    <Form onSubmit={submit} validate={validate}>
       {({ handleSubmit, submitFailed, errors, values }) => (
         <StyledForm onSubmit={handleSubmit}>
           {meldingTilBehandler.isSuccess && (
             <Alert variant="success" size="small">
               {`Meldingen ble sendt ${tilDatoMedManedNavnOgKlokkeslett(now)}`}
             </Alert>
           )}
-          {isBehandlerdialogLegeerklaringEnabled && (
-            <MeldingsType>
-              <SelectMeldingType />
-              {values.type && <MeldingsTypeInfo meldingType={values.type} />}
-            </MeldingsType>
-          )}
+          <MeldingsType>
+            <SelectMeldingType />
+            {values.type && <MeldingsTypeInfo meldingType={values.type} />}
+          </MeldingsType>
           <VelgBehandler
             selectedBehandler={selectedBehandler}
             setSelectedBehandler={

diff --git a/src/data/unleash/unleash_types.ts b/src/data/unleash/unleash_types.ts
@@ -5,14 +5,12 @@ export type Toggles = {
 // See toggles: https://teamsykefravr-unleash-web.nav.cloud.nais.io/features
 export enum ToggleNames {
   isVirksomhetsinputEnabled = "isVirksomhetsinputEnabled",
-  isBehandlerDialogLegeerklaringEnabled = "isBehandlerDialogLegeerklaringEnabled",
-  isReturLegeerklaringEnabled = "isReturLegeerklaringEnabled",
+  isMeldingsTypeMeldingFraNAVEnabled = "isMeldingsTypeMeldingFraNAVEnabled",
   isMotebehovTilbakemeldingEnabled = "isMotebehovTilbakemeldingEnabled",
 }
 
 export const defaultToggles: Toggles = {
   isVirksomhetsinputEnabled: false,
-  isBehandlerDialogLegeerklaringEnabled: false,
-  isReturLegeerklaringEnabled: false,
+  isMeldingsTypeMeldingFraNAVEnabled: false,
   isMotebehovTilbakemeldingEnabled: false,
 };
diff --git a/test/behandlerdialog/MeldingTilBehandlerTest.tsx b/test/behandlerdialog/MeldingTilBehandlerTest.tsx
@@ -17,12 +17,6 @@ import {
   expectedLegeerklaringDocument,
   expectedTilleggsopplysningerDocument,
 } from "./testDataDocuments";
-import { unleashQueryKeys } from "@/data/unleash/unleashQueryHooks";
-import {
-  BEHANDLENDE_ENHET_DEFAULT,
-  VEILEDER_IDENT_DEFAULT,
-} from "../../mock/common/mockConstants";
-import { mockUnleashResponse } from "../../mock/unleashMocks";
 
 let queryClient: QueryClient;
 
@@ -45,31 +39,6 @@ describe("MeldingTilBehandler", () => {
     queryClient = queryClientWithMockData();
   });
 
-  it("Viser overskrift og warning-alert for bruk av tilleggsopplysninger hvis toggle for legeerklæring er av", () => {
-    queryClient.setQueryData(
-      unleashQueryKeys.toggles(
-        BEHANDLENDE_ENHET_DEFAULT.enhetId,
-        VEILEDER_IDENT_DEFAULT
-      ),
-      () => {
-        return {
-          ...mockUnleashResponse,
-          isBehandlerDialogLegeerklaringEnabled: false,
-        };
-      }
-    );
-
-    renderMeldingTilBehandler();
-
-    const tilleggsopplysningerInfoText =
-      "Her kan du kun be om tilleggsopplysninger med takst L8. Dialogmeldingen skal bare benyttes i sykefraværsoppfølgingen. Meldingen vises også til innbyggeren på Min side.";
-
-    expect(screen.getByRole("heading", { name: "Skriv til behandler" })).to
-      .exist;
-    expect(screen.getByRole("img", { name: "Advarsel" })).to.exist;
-    expect(screen.getByText(tilleggsopplysningerInfoText)).to.exist;
-  });
-
   it("Viser overskrift og warning-alert for kopi til bruker hvis toggle for legeerklæring er på", () => {
     renderMeldingTilBehandler();