1481 implement similar positions on recruitmentapplicationformpage #1596
Conversation
…for applicant serializer
There was a problem hiding this comment.
Kan kanskje være litt forvirrende om navn og tags ikke stemmer? Ikke så farlig for meg assa
There was a problem hiding this comment.
Skjønner ikke helt hva du mener. Samme "kategori" av tags har en "stillings navn prefik", så det skal ikke bli mismatch mellom navn og tag.
| recruitment_id = self.kwargs.get('id') | ||
| tags_param = self.request.query_params.get('tags') |
There was a problem hiding this comment.
Funker det ikke å bruke getlist her?
There was a problem hiding this comment.
Hvis jeg forstår det riktig må klienten sende queryparams på denne måten: ?tags=tag1&tags=tag2&tags=tag3 hvis man skal bruke getlist
Gitt at tags i denne implementasjonene er lagret i en komma separert string må man uansett formatere dataen på et elle annet tidspunkt. Om man skulle brukt getlist må man gi API kall funksjonen tags: string[]
og formatere url med params på denne måten:
const tagsQuery = tags.map(t => `tags=${encodeURIComponent(t)}`).join('&');
const url = `${BACKEND_DOMAIN}/positions?id=${recruitmentId}&${tagsQuery}&position_id=${currentPositionId}`;
Jeg tror get er den enkleste måten å gjøre det på siden tags er i en string separert med komma, men hvis noe av det over virker som at jeg ikke har forstått forskjellen på get og getlist så si ifra.
There was a problem hiding this comment.
For å erstatte Samf3 funksjonaliteten tror jeg dette er den enkleste måten å gjøre det på
frontend/src/routes/backend.ts
Outdated
| } as const; No newline at end of file | ||
| } as const; |
There was a problem hiding this comment.
fixed. Noe rart med generering av routes tror jeg
| {recruitmentPositionsForGang && recruitmentPositionsForGang.length > 0 && ( | ||
| <div className={styles.other_positions}> | ||
| <h2 className={styles.sub_header}> | ||
| {t(KEY.recruitment_otherpositions)} {dbT(recruitmentPosition?.gang, 'name')} | ||
| </h2> | ||
| {recruitmentPositionsForGang.map((pos) => ( | ||
| <Button | ||
| key={pos.id} | ||
| display="pill" | ||
| theme="outlined" | ||
| onClick={() => { | ||
| navigate({ | ||
| url: reverse({ | ||
| pattern: ROUTES.frontend.recruitment_application, | ||
| urlParams: { positionId: pos.id, gangId: pos.gang.id }, | ||
| }), | ||
| }); | ||
| }} | ||
| > | ||
| {dbT(pos, 'name')} | ||
| </Button> | ||
| ))} | ||
| </div> | ||
| )} |
There was a problem hiding this comment.
Dette kan flyttes ut i en konstant
There was a problem hiding this comment.
Done! Kunne sikkert lagd en helt egen komponent av det, men planen er at det skal styles bedre i et annet issue uansett.
What
Created backend and frontend for fetching positions from tags. This makes it possible to fetch positions similar to each other.
Modified seed script to create more realisitc tags.
Fixes some rendering issues on RecruitmentApplicationFromPage
Bette styling will be done in another PR. The entire page should get a glow-up.
Why
Fetching similar positions is a feature on Samf3
closes #1481