From ad2fe865c555d8cd71eb18ec96e0600a8ee2ecb1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Solg=C3=A5rd?= Date: Tue, 29 Aug 2023 10:01:31 +0200 Subject: [PATCH] IS-1488: Removed unused toggles + added new one --- server/unleash.ts | 10 ++---- .../meldinger/MeldingerISamtale.tsx | 7 +---- .../MeldingTilBehandler.tsx | 27 ++++------------ .../MeldingTilBehandlerSkjema.tsx | 28 ++++------------- src/data/unleash/unleash_types.ts | 6 ++-- .../MeldingTilBehandlerTest.tsx | 31 ------------------- 6 files changed, 18 insertions(+), 91 deletions(-) diff --git a/server/unleash.ts b/server/unleash.ts index 5f8857d27..0b301e9bd 100644 --- 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 index 790a31c33..8bb146c18 100644 --- 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,7 +58,6 @@ const MeldingFraBehandler = ({ meldinger, melding, }: MeldingFraBehandlerProps) => { - const { toggles } = useFeatureToggles(); const isLegeerklaring = melding.type === MeldingType.FORESPORSEL_PASIENT_LEGEERKLARING; const sentReturForLegeerklaring = meldinger.some( @@ -67,10 +65,7 @@ const MeldingFraBehandler = ({ m.type === MeldingType.HENVENDELSE_RETUR_LEGEERKLARING && m.parentRef === melding.uuid ); - const showReturLegeerklaring = - toggles.isReturLegeerklaringEnabled && - isLegeerklaring && - !sentReturForLegeerklaring; + const showReturLegeerklaring = isLegeerklaring && !sentReturForLegeerklaring; return ( diff --git a/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandler.tsx b/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandler.tsx index 8d7b86a18..34aed5f9d 100644 --- 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 ( <> {texts.header} - {isLoading ? ( - - ) : ( - <> - - {toggles.isBehandlerDialogLegeerklaringEnabled - ? texts.meldingVisesTilBruker - : texts.tilleggsopplysningerInfo} - - - - )} + + {texts.alertInfo} + + ); }; diff --git a/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandlerSkjema.tsx b/src/components/behandlerdialog/meldingtilbehandler/MeldingTilBehandlerSkjema.tsx index 0bd96faec..099f4f23c 100644 --- 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(); @@ -128,15 +122,7 @@ export const MeldingTilBehandlerSkjema = ({ }; return ( -
+ {({ handleSubmit, submitFailed, errors, values }) => ( {meldingTilBehandler.isSuccess && ( @@ -144,12 +130,10 @@ export const MeldingTilBehandlerSkjema = ({ {`Meldingen ble sendt ${tilDatoMedManedNavnOgKlokkeslett(now)}`} )} - {isBehandlerdialogLegeerklaringEnabled && ( - - - {values.type && } - - )} + + + {values.type && } + { 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();