Janzz (#249)

* Janzz-element for å søke opp janz-stilling * Deploy to dev * Bytt ut styrk-felt med janzz * Manglet PAM_ONTOLOGI_URL * pam-ontologi-kall krever Nav-CallId * crypto ikke tilgjengelig, men det er uuid * Sjekk melding fra azure om hvorfor det er feil * Må lese ut av body * Må lese ut av body * Må lese ut av body * Må lese ut av body * Må lese ut av body * Må lese ut av body * Må lese ut av body * Kall til pam-ontologi ser ikke ut til å kreve token * Kallet skal være en GET, ikke en delete * pam-ontologi må være i outbound for service-discovery * Lagre Janzz i stilling i stedet for styrk * Feil i adValidationReducer.ts * Fjern Styrk-referanser * Mangler css * Feil i referanse til css-fil * Initiell verdi på stilling * bygg-feil på grunn av mock * Legg til lokal mocking av pam-ontologi * categoryList skal være en array, ikke en objekt * Dispatch bare endring på categoryList om ny yrkestittel er valgt * Fiks mock * Les tittel-felt istedet for styrkEllerTittel * Bruk tittel-felt hvis tilgjengelig * Default mocktittel på stilling * Bruk tittel-felt alltid * Fjern styrkEllerTittel til fordel for det nye tittel-feltet * fiks metode for å hente tittel, skal ikke ha default verdi * Janzz felt skal kunne tømmes helt * Janzz felt skal ikke under opprettelse ha stilling uten valgt jobbtittel * Janzz felt skal ikke under opprettelse ha stilling uten valgt jobbtittel * Legg på validering på janzz-feltet * Legg på validering av janzz med feilmelding * Bytt til combobox * Fiks feil med at ingenting kan skrives inn * Prøv med typeahead for yrke på første bokstav * Refactor løsning * Sett til minimum to bokstaver i yrkestypeahead * Lag use metode for janzz titler * Fiks url mot ontologi * Fiks url ontologi * Legg på headers slik at vi kan kalle arbeidsplassen for janzz kall * Se om det går fint uten å sende unndefined janzz ved sletting av ykestittel * FIlterer bort jobbmessetittel, og legg på påkrevd label for yrkestittel * Legg inn nyhetssak for yrkestittel * Legg til filteroptions --------- Co-authored-by: Joar Aurdal <[email protected]>
navikt · Nov 1, 2024 · de0da31 · de0da31
1 parent f197306
commit de0da31
Show file tree

Hide file tree

Showing 33 changed files with 264 additions and 749 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -45,7 +45,7 @@ jobs:
     deploy-til-dev:
         name: Deploy til dev-gcp
         needs: bygg-og-push-docker-image
-        if: github.event_name == 'workflow_dispatch' || github.ref == 'refs/heads/master' || github.ref == 'refs/heads/finn-kandidat-for-stilling'
+        if: github.event_name == 'workflow_dispatch' || github.ref == 'refs/heads/master' || github.ref == 'refs/heads/janzz'
         runs-on: ubuntu-latest
         steps:
             - uses: actions/checkout@v3

diff --git a/deploy/nais.yaml b/deploy/nais.yaml
@@ -50,6 +50,8 @@ spec:
           - application: presenterte-kandidater-api
           - application: modiacontextholder
             namespace: personoversikt
+          - application: pam-ontologi
+            namespace: teampam
         external:
           - host: {{ statistikkApiHost }}
           - host: {{ kandidatApiHost }}
@@ -91,3 +93,5 @@ spec:
           value: http://modiacontextholder.personoversikt
         - name: KANDIDATVARSEL_API_URL
           value: http://rekrutteringsbistand-kandidatvarsel-api
+        - name: PAM_ONTOLOGI_URL
+          value: http://pam-ontologi.teampam
diff --git a/mock/stilling-api/mockStilling.ts b/mock/stilling-api/mockStilling.ts
@@ -48,6 +48,7 @@ export const mockStilling /* : Stilling */ = {
     mediaList: [],
     contactList: [],
     title: 'TITTEL FRA ARBEIDSPLASSEN',
+    tittel: 'Tittel på stilling',
     status: Status.Aktiv,
     privacy: Privacy.Intern,
     source: Kilde.Intern,
@@ -71,6 +72,7 @@ export const mockStilling /* : Stilling */ = {
     categoryList: [
         {
             name: 'Kokk',
+            code: '1234',
             categoryType: 'STYRK08NAV',
         },
     ],
@@ -391,7 +393,7 @@ export const mockFormidling = {
                 phone: '',
             },
         ],
-        styrkEllerTittel: 'Formidling',
+        tittel: 'Formidling',
     },
     stillingsinfo: {
         eierNavident: null,
@@ -407,7 +409,7 @@ export const mockFormidlingMin = {
     ...mockFormidling,
     stilling: {
         ...mockFormidling.stilling,
-        styrkEllerTittel: 'Formidling MIN',
+        tittel: 'Formidling MIN',
         uuid: 'minFormidling',
         administration: {
             ...mockFormidling.stilling.administration,
@@ -425,7 +427,7 @@ export const mockStoppetStilling = {
     ...mockStilling,
     stilling: {
         ...mockFormidling.stilling,
-        styrkEllerTittel: 'Stoppet stilling',
+        tittel: 'Stoppet stilling',
         uuid: 'stoppet',
         status: Status.Stoppet,
         // administration: {

diff --git a/mock/stillingssok-proxy/mockEsStilling.ts b/mock/stillingssok-proxy/mockEsStilling.ts
@@ -12,7 +12,7 @@ export const mockEsRekrutteringsbistandstilling: EsRekrutteringsbistandstilling
     stilling: {
         ...mockStilling,
         uuid: 'intern',
-        styrkEllerTittel: 'Intern stilling',
+        tittel: 'Intern stilling',
         annonsenr: String(mockStilling.id),
         categories: [
             {
@@ -44,7 +44,7 @@ export const mockEsRekrutteringsbistandstillingMin: EsRekrutteringsbistandstilli
     stilling: {
         ...mockEsRekrutteringsbistandstilling.stilling,
         uuid: 'minIntern',
-        styrkEllerTittel: 'Intern stilling MIN',
+        tittel: 'Intern stilling MIN',
         administration: {
             ...(mockEsRekrutteringsbistandstilling.stilling.administration as Administration),
             navIdent: 'Z123456',
@@ -63,7 +63,7 @@ export const mockEsRekrutteringsbistandstillingEkstern: EsRekrutteringsbistandst
         uuid: 'ekstern',
         source: Kilde.Finn,
         medium: Medium.Ass,
-        styrkEllerTittel: 'Ekstern stilling',
+        tittel: 'Ekstern stilling',
         annonsenr: String(mockStilling.id),
         categories: [
             {
@@ -96,7 +96,7 @@ export const mockEsRekrutteringsbistandstillingEksternMin: EsRekrutteringsbistan
     stilling: {
         ...mockEsRekrutteringsbistandstillingEkstern.stilling,
         uuid: 'minEkstern',
-        styrkEllerTittel: 'Ekstern stilling MIN',
+        tittel: 'Ekstern stilling MIN',
         administration: {
             ...(mockEsRekrutteringsbistandstillingEkstern.stilling
                 .administration as Administration),

diff --git a/server/src/onBehalfOfToken.ts b/server/src/onBehalfOfToken.ts
@@ -65,8 +65,8 @@ async function hentNyttOnBehalfOfToken(accessToken: string, scope: string): Prom
         return body as OboToken;
     } else {
         logger.info(
-            `Klarte ikke å hente on behalf of token for scope "${scope}", fikk status ${response.status} (${response.statusText}) årsak: `,
-            body
+            `Klarte ikke å hente on behalf of token for scope "${scope}", fikk status ${response.status} (${response.statusText}) årsak: ` +
+                JSON.stringify(body, null, 2)
         );
 
         throw response;

diff --git a/server/src/server.ts b/server/src/server.ts
@@ -31,6 +31,7 @@ const scopes = {
     presenterteKandidater: `api://${cluster}.toi.presenterte-kandidater-api/.default`,
     kandidatsøk: `api://${cluster}.toi.rekrutteringsbistand-kandidatsok-api/.default`,
     kandidatvarsel: `api://${cluster}.toi.rekrutteringsbistand-kandidatvarsel-api/.default`,
+    pamOntologi: `api://${cluster}.teampam.pam-ontologi/.default`,
 };
 
 const {
@@ -45,6 +46,7 @@ const {
     PRESENTERTE_KANDIDATER_API,
     MODIA_CONTEXT_HOLDER_API,
     KANDIDATVARSEL_API_URL,
+    PAM_ONTOLOGI_URL,
 } = process.env;
 
 const startServer = () => {
@@ -64,6 +66,7 @@ const startServer = () => {
     proxyMedOboToken('/stilling-api', STILLING_API_URL, scopes.stilling);
     proxyMedOboToken('/kandidat-api', KANDIDAT_API_URL, scopes.kandidat);
     proxyMedOboToken('/kandidatsok-api', KANDIDATSOK_API_URL, scopes.kandidatsøk);
+    proxyUtenToken('/pam-ontologi', PAM_ONTOLOGI_URL);
     proxyMedOboToken('/kandidatvarsel-api', KANDIDATVARSEL_API_URL, scopes.kandidatvarsel);
     proxyMedOboToken(
         '/foresporsel-om-deling-av-cv-api',

diff --git a/src/api/api.msw.ts b/src/api/api.msw.ts
@@ -3,11 +3,13 @@ import { foresporselApiMock } from './foresporsel-om-deling-av-cv-api/mock.msw';
 import { kandidatApiMock } from './kandidat-api/mock.msw';
 import { kandidatSokApiMock } from './kandidat-søk-api/mock.msw';
 import { statistikkApiMock } from './statistikk-api/mock.msw';
+import { pamOntologiMock } from './pam-ontologi/mock.msw';
 
 export const apiMockHandlers = [
     ...kandidatApiMock,
     ...foresporselApiMock,
     ...statistikkApiMock,
     ...kandidatSokApiMock,
     ...stillingApiMock,
+    ...pamOntologiMock,
 ];
diff --git a/src/api/fetcher.ts b/src/api/fetcher.ts
@@ -2,17 +2,23 @@ import { z, ZodSchema } from 'zod';
 
 const basePath = import.meta.env.VITEST ? 'http://localhost:3000' : '';
 
-export const getAPIwithSchema = <T>(schema: ZodSchema<T>): ((url: string) => Promise<T>) => {
+export const getAPIwithSchema = <T>(
+    schema: ZodSchema<T>,
+    headers?: HeadersInit
+): ((url: string) => Promise<T>) => {
     return async (url: string) => {
-        const data = await getAPI(url);
+        const data = await getAPI(url, headers);
         return schema.parse(data);
     };
 };
 
-export const getAPI = async (url: string) => {
+export const getAPI = async (url: string, headers?: HeadersInit) => {
     const response = await fetch(basePath + url, {
         method: 'GET',
         credentials: 'include',
+        headers: {
+            ...headers,
+        },
     });
 
     if (response.ok) {

diff --git a/src/api/pam-ontologi/mock.msw.ts b/src/api/pam-ontologi/mock.msw.ts
@@ -0,0 +1,3 @@
+import { pamOntologiMockMsw } from './pamOntologi';
+
+export const pamOntologiMock = [pamOntologiMockMsw];
diff --git a/src/api/pam-ontologi/pamOntologi.ts b/src/api/pam-ontologi/pamOntologi.ts
@@ -0,0 +1,37 @@
+/**
+ * Endepunkt /statistikk
+ */
+import { HttpResponse, http } from 'msw';
+import { z } from 'zod';
+
+export const pamOntologiEndepunkt = `/pam-ontologi/rest/typeahead/stilling*`;
+
+const janzzKategoriDTOSchema = z.object({
+    konseptId: z.number(),
+    styrk08: z.string(),
+    esco: z.string(),
+    escoLabel: z.string(),
+    label: z.string(),
+    undertype: z.string(),
+});
+
+export type janzzKategoriDTO = z.infer<typeof janzzKategoriDTOSchema>;
+
+const janzzKategoriMock = (): janzzKategoriDTO[] | null => {
+    return [
+        {
+            konseptId: 358507,
+            styrk08: '1112',
+            esco: 'http://data.europa.eu/esco/isco/C1112',
+            escoLabel: 'Toppledere i offentlig administrasjon',
+            label: 'Departementsråd',
+            undertype: '',
+        },
+    ];
+};
+
+export const pamOntologiMockMsw = http.get(pamOntologiEndepunkt, ({ request }) => {
+    const janzzKategorier = janzzKategoriMock();
+
+    return HttpResponse.json(janzzKategorier);
+});
diff --git a/src/api/stillings-api/hentJanzzyrker.ts b/src/api/stillings-api/hentJanzzyrker.ts
@@ -0,0 +1,36 @@
+import useSWR from 'swr';
+import { getAPIwithSchema } from '../fetcher';
+import { z } from 'zod';
+import { api } from 'felles/api';
+import { v4 as uuidv4 } from 'uuid';
+
+export const hentJanzzYrkerEndepunkt = `${api.pamOntologi}/rest/typeahead/stilling`;
+
+export const janzzStillingSchema = z.object({
+    label: z.string(),
+    konseptId: z.number(),
+    styrk08: z.any().optional(),
+    esco: z.any().optional(),
+    escoLabel: z.any().optional(),
+    undertype: z.any().optional(),
+});
+
+const hentJanzzYrkerSchema = z.array(janzzStillingSchema);
+
+export type JanzzStillingDTO = z.infer<typeof janzzStillingSchema>;
+export type HentJanzzYrkerDTO = z.infer<typeof hentJanzzYrkerSchema>;
+
+export const useHentJanzzYrker = (query: string) => {
+    const shouldFetch = query.length > 1;
+    const endpoint = shouldFetch
+        ? `${hentJanzzYrkerEndepunkt}?stillingstittel=${encodeURIComponent(query)}`
+        : null;
+
+    const headers = {
+        'X-Requested-With': 'XMLHttpRequest',
+        'Cache-Control': 'no-cache, no-store',
+        'Nav-CallId': uuidv4(),
+    };
+
+    return useSWR(endpoint, getAPIwithSchema(hentJanzzYrkerSchema, headers));
+};
diff --git a/src/dev/DevMockModal.tsx b/src/dev/DevMockModal.tsx
@@ -20,6 +20,7 @@ import { stillingsapiMock } from '../api/stillings-api/mock.msw';
 import DevMockApi from './DevMockApi';
 import DevRoller from './DevMockMeg';
 import style from './DevTools.module.css';
+import { pamOntologiMock } from '../api/pam-ontologi/mock.msw';
 
 export interface IDevMockModal {
     children?: React.ReactNode | undefined;
@@ -85,6 +86,10 @@ const mockConfig = [
         navn: 'Stilling api',
         mock: stillingsapiMock,
     },
+    {
+        navn: 'Pam Ontologi',
+        mock: pamOntologiMock,
+    },
 ];
 
 const DevMockModal: React.FC<IDevMockModal> = ({ children }) => {

diff --git a/src/felles/api.ts b/src/felles/api.ts
@@ -12,6 +12,7 @@ export const api = {
     modiaContextHolder: '/modiacontextholder/api',
     arbeidsgiverNotifikasjon: '/arbeidsgiver-notifikasjon-api',
     kandidatSokApi: '/kandidatsok-api',
+    pamOntologi: '/pam-ontologi',
 };
 
 export const videresendTilInnlogging = () => {

diff --git a/src/felles/domene/stilling/EsStilling.ts b/src/felles/domene/stilling/EsStilling.ts
@@ -23,7 +23,7 @@ export type EsStilling = Stillingbase & {
     contacts: Kontaktinfo[];
     employer: EsArbeidsgiver | null;
     properties: EsProperties;
-    styrkEllerTittel: string;
+    tittel: string;
 };
 
 export type EsProperties = Partial<{

diff --git a/src/felles/domene/stilling/Stilling.ts b/src/felles/domene/stilling/Stilling.ts
@@ -43,24 +43,12 @@ export type Stilling = Stillingbase & {
     };
 };
 
-// TODO: Fjern nå vi har byttet om
-export const USE_STYRK_AS_TITLE_FEATURE_TOGGLE = true;
 export const hentTittelFraStilling = (stilling: Stilling) => {
-    if (!USE_STYRK_AS_TITLE_FEATURE_TOGGLE) {
-        return stilling.title;
-    }
-
     if (stilling.source !== 'DIR') {
         return stilling.title;
     }
 
-    const passendeStyrkkoder =
-        stilling.categoryList?.filter(({ categoryType }) => categoryType === 'STYRK08NAV') ?? [];
-
-    if (passendeStyrkkoder.length === 0) {
-        return 'Stilling uten valgt jobbtittel';
-    }
-    return passendeStyrkkoder.map((it) => it.name).join('/');
+    return stilling.categoryList?.find(({ categoryType }) => categoryType === 'JANZZ')?.name || '';
 };
 
 export enum System {
@@ -122,6 +110,7 @@ export type EsStyrkCategory = {
 
 export type StyrkCategory = {
     categoryType: string;
+    code: string;
     name: string;
 };
 

diff --git a/src/felles/header/nyheter/nyhetssaker.tsx b/src/felles/header/nyheter/nyhetssaker.tsx
@@ -4,6 +4,21 @@ const tilDato = (dag: number, måned: number, år: number, timer = 0, minutter =
     new Date(år, måned - 1, dag, timer, minutter);
 
 const nyhetssaker: Nyhet[] = [
+    {
+        dato: tilDato(1, 11, 2024),
+        tittel: 'STYRK byttet ut med yrkestittel i stillingsregistreringen',
+        innhold: (
+            <>
+                <p>Vi har byttet ut STYRK i stillingsregistreringen.</p>
+                <p>
+                    Vi har byttet ut STYRK i stillingsregistreringen til en bedre løsning som lar
+                    deg velge mer detaljerte og treffsikre stillingstyper. Valgt yrkestittel vil bli
+                    tittel på stillingen. Yrkestittel setter du i feltet med overskriften
+                    «Yrkestittel som vises på stillingen».
+                </p>
+            </>
+        ),
+    },
     {
         dato: tilDato(13, 6, 2024),
         tittel: 'Ny tilgangsstyring',

diff --git a/src/stilling/api/api.ts b/src/stilling/api/api.ts
@@ -9,6 +9,7 @@ import Stilling, {
 import { Miljø, getMiljø } from 'felles/miljø';
 import { fetchGet, fetchPost, fetchPut } from './apiUtils';
 import devVirksomheter from './devVirksomheter';
+import { v4 as uuidv4 } from 'uuid';
 
 export const postStilling = async (
     stilling: Partial<Stilling>,
@@ -129,6 +130,35 @@ export const fetchOrgnrSuggestions = async (orgnummer: string): Promise<Enhetsre
     ];
 };
 
+export type JanzzStilling = {
+    konseptId: number;
+    label: string;
+};
+
+export const fetchJanzzYrker = async (typeahead: string): Promise<JanzzStilling[]> => {
+    const response = await fetch(
+        `${api.pamOntologi}/rest/typeahead/stilling?stillingstittel=${typeahead}`,
+        {
+            method: 'GET',
+            credentials: 'include',
+            headers: {
+                'X-Requested-With': 'XMLHttpRequest',
+                'Cache-Control': 'no-cache, no-store',
+                'Nav-CallId': uuidv4(),
+            },
+        }
+    );
+    const result = await response.json();
+    return result.map((janzzStilling: any) => ({
+        konseptId: janzzStilling.konseptId,
+        styrk08: janzzStilling.styrk08,
+        esco: janzzStilling.esco,
+        escoLabel: janzzStilling.escoLabel,
+        label: janzzStilling.label,
+        undertype: janzzStilling.undertype,
+    }));
+};
+
 /**
  * TODO: Dette er en workaround, fordi det finnes annonser med ad.administration=null i databasen.
  * Når databasen er migrert og ikke inneholder administration=null kan denne workarounden fjernes.