Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sykmelding bistand duplikat #1619

Merged
merged 3 commits into from
Jan 28, 2025
Merged

Conversation

geir-waagboe
Copy link
Contributor

@geir-waagboe geir-waagboe commented Jan 23, 2025

Hva har blitt lagt til✨🌈

Markerer behandler-ber-om-bistand-oppgaver fra sykmeldinger som duplikate hvis samme tekst har kommet i en tidligere sykmelding.

Screenshots 📸✨

dupl

@geir-waagboe geir-waagboe requested a review from a team as a code owner January 23, 2025 14:16
@geir-waagboe geir-waagboe force-pushed the sykmelding-bistand-duplikat branch from 280f8bb to 04ac879 Compare January 23, 2025 14:17
Copy link
Contributor

@andersrognstad andersrognstad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Koden ser fin ut, men mulig det blir noe styling-endringer her (f.eks. flytte Tag), ref diskusjon på Slack.

@geir-waagboe
Copy link
Contributor Author

Koden ser fin ut, men mulig det blir noe styling-endringer her (f.eks. flytte Tag), ref diskusjon på Slack.

Forsøkt meg med en oppdatering nå.

<Heading size="medium" level="2">
{texts.header}
</Heading>
<Tag variant="warning-moderate">Mulig duplikat</Tag>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kan evnt bytte ut med en "blekere" variant ved å bruke "warning".

Comment on lines 52 to 75
<div className={"flex flex-row justify-between"}>
<Heading size="medium" level="2">
{texts.header}
</Heading>
<Tag variant="warning-moderate">Mulig duplikat</Tag>
<HelpText
placement="left"
title="Informasjon fra felter i sykmeldingen fra behandler"
>
{texts.helptextDuplicate}
</HelpText>
</div>
) : (
<div className={"flex flex-row justify-between"}>
<Heading size="medium" level="2">
{texts.header}
</Heading>
<HelpText
title="Informasjon fra felter i sykmeldingen fra behandler"
placement="left"
>
{texts.helptext}
</HelpText>
</div>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tror du kan forenkle ved å flytte sjekkene for duplikat "inn" akkurat der de trengs, på denne måten:

<div className={"flex flex-row justify-between"}>
        <Heading size="medium" level="2">
          {texts.header}
        </Heading>
        <div className="flex gap-2 items-center">
          {!!oppgave.duplikatReferanseUuid && (
            <Tag variant="warning-moderate">Mulig duplikat</Tag>
          )}
          <HelpText
            placement="bottom"
            title="Se mer informasjon om bistandsbehovet"
          >
            {!!oppgave.duplikatReferanseUuid
              ? texts.helptextDuplicate
              : texts.helptext}
          </HelpText>
        </div>
      </div>

Ved å wrappe Tag og HelpText i en egen div, kan du plassere de begge helt i hjørnet pga justify-between på parent-diven. Og så innenfor den diven legge på avstand mellom de. Og så endret jeg placement="bottom" sånn at de ikke kræsjer. Endra også title på HelpText som Peter reagerte på i tråden.

Blir sånn her:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fikset nå (tror jeg)

@geir-waagboe geir-waagboe force-pushed the sykmelding-bistand-duplikat branch from d6732e7 to 4170ad4 Compare January 27, 2025 09:13
@andersrognstad
Copy link
Contributor

Har vi forresten en test på den VurderBistandsbehov-komponenten?

@andersrognstad
Copy link
Contributor

Har vi forresten en test på den VurderBistandsbehov-komponenten?

La til en test 🤓

@geir-waagboe geir-waagboe merged commit 9b5d489 into master Jan 28, 2025
3 checks passed
@geir-waagboe geir-waagboe deleted the sykmelding-bistand-duplikat branch January 28, 2025 14:41
vetlesolgaard pushed a commit that referenced this pull request Jan 29, 2025
vetlesolgaard pushed a commit that referenced this pull request Jan 29, 2025
vetlesolgaard added a commit that referenced this pull request Jan 31, 2025
* IS-2999-3: Be om oppfolgingsplan design finpuss

* Sykmelding bistand duplikat (#1619)

* IS-2999-3: Be om oppfolgingsplan design

---------

Co-authored-by: Geir Arne Waagbø <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants