-
Notifications
You must be signed in to change notification settings - Fork 0
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
OK-436 #53
OK-436 #53
Conversation
onChange: (e: SelectChangeEvent) => void; | ||
}) => { | ||
const { t } = useTranslation(); | ||
// eslint-disable-next-line @typescript-eslint/no-unused-vars |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Miksi? Et näytä käyttävän erroria ja refetchiä tos, etkö voi vaan jättää ne pois?
margin: theme.spacing(0.5, 0), | ||
})); | ||
|
||
const EMPTY_ARRAY: Array<unknown> = []; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tästä vois tehdä constantin jos koet tarvitsevasi useammin. Myös jos on tarkoitus ettei tuota voi mutiloida niin Petteri otti käyttöön meidän projussa Object.freeze([]) as Array;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tuo oli itse asiassa teidän projektista apinoitu mut ei tainnut sit olla tuorein versio käsillä 😄 ja pistin sen nyt toistaiseksi tuohon vakioks kun ei ollut muualla vielä käyttöä
@@ -23,7 +23,8 @@ import { SearchParams } from 'nuqs/server'; | |||
import { searchParamsCache } from '@/app/lib/searchParams'; | |||
import { initTranslations } from '@/app/i18n/localization'; | |||
/* eslint-disable @typescript-eslint/no-explicit-any */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voiko tän poistaa?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Näköjään voi
@@ -227,3 +230,25 @@ export async function searchOrganisaatio(searchStr: string): Promise<Organisaati | |||
} | |||
return res.json(); | |||
} | |||
|
|||
export async function fetchLahettavatPalvelut(): Promise<string[]> { | |||
const sessionCookie = cookies().get(cookieName); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tätä login toiminnallisuutta vois handlata tekemällä oman fetch funktion joka syövereissänsä kutsuu tuota oikeata fetchiä. Sillä saat saman kirjautumispäättely toiminnallisuuden kaikkialle.
…palvelujen hakuun
… ja lisätty retry ym
…aan usequery jotta vältetään initial render-ongelmat
Listaus lähettävistä palveluista bäkkäriin.
Lähettävät palvelut käliin hakukriteeriksi.
Pohjana branch OK-606_haku joka ei ole vielä mainissa, koska hakutoiminnot sisältävät kantamuutoksia joita vielä testataan QA:lla ja vietävä hallitusti tuotantoon.