Implement stop status to effect on routes and lines#1454
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Huulivoide
left a comment
There was a problem hiding this comment.
@Huulivoide reviewed 7 files and all commit messages, and made 11 comments.
Reviewable status: 7 of 26 files reviewed, 11 unresolved discussions (waiting on PasiVuohijoki).
test-db-manager/src/types/enums.ts line 29 at r1 (raw file):
TrunkLineStop = 'trunkLineStop', Mirrors = 'mirrors', StopStateValidityStart = 'stopStateValidityStart',
Tää näyttää aika pelottavalta näin parina ekana rivinä, jotka PR:ssä näkyy 🙈
Joo, elikäs nää lähtee pois ja sitä myöten kaikki näihin liittyvä koodi. Ja sit jos sitä muutosta halutaan aikatauluttaa, niin se pitää hoitaa sitten siten että pysäkistä luodaan kopio, jossa se muutos on. MVP ratkasu, käyttäjä luo itse sen kopion eka ja asettaa uuden tilan sitten siihen.
Oikeat käyttötapaukset:
- Väliaikasesti pois käytöstä. → Luodaan uus Väliaikainen prio versio pysäkistä, joka pistetään pois päältä.
- Lopullisesti pois käytöstä. → Päätetään nykynen perus versio, luodaan uus perusversio tyhjällä päättymispäivällä, merkitään se pois käytöstä.
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 38 at r1 (raw file):
type UseStopStateChangeConfirmationParams< TFormState extends StopStateFormFields,
Laitan muistin aikaan jo kommentitn ylös. Mutta tarviiks tä oikeesti olla geneerinen? 🤔
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 40 at r1 (raw file):
TFormState extends StopStateFormFields, > = { readonly currentStopState: StopPlaceState | null | undefined;
Tän pitä olla aina pakoillaan. Ainakaan enää ei voi tallentaa pysäkkiä jos sillä ei ole statea, tietokanta rajotukset alkaa huutaan.
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 75 at r1 (raw file):
); const resolveStopPointId = async (): Promise<string | undefined> => {
Tän vois nostaa ylemmäks erilliseks hookki funkkariks, koska tää useStopStateChangeConfirmation on niin pitkä
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 79 at r1 (raw file):
return scheduledStopPointId; } if (!stopPlaceRef) {
Tän voi olettaa olevan aina olemassa.
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 89 at r1 (raw file):
}; const getAffectedRoutes = async (): Promise<
Sama, tai oikeestaan nää vois olla samassa, koska tää on ton edellisen ainoo käyttöpaikka
ui/src/components/stop-registry/stops/stop-details/useStopStateChangeConfirmation.ts line 100 at r1 (raw file):
}); const stopData = mapStopResultToStop(result); if (!stopData) {
Tän iffin voi pudottaa pois jos alemmas pistää stopData?.schedu...
ui/src/components/stop-registry/stops/stop-details/basic-details/BasicDetailsSection.tsx line 38 at r1 (raw file):
timingPlaceId: stop.quay?.timingPlaceId ?? stop.timing_place_id ?? null, stopState: stop.quay?.stopState ?? undefined, stopStateValidityStart: stop.quay?.stopStateValidityStart
Pysäkin tilalla ei pitäs olla mitään erillistä voimasaoloa. Se on se pysökin voimassa olo
ui/src/components/stop-registry/stops/stop-details/basic-details/BasicDetailsSection.tsx line 62 at r1 (raw file):
const { observationDate } = useObservationDateQueryParam(); const effectiveStopState = getEffectiveStopState(
Tää on kokonaan tarpeeton. Pysäkin voimassaolo on se mikä se on.
ui/src/components/stop-registry/stops/stop-details/basic-details/BasicDetailsSection.tsx line 82 at r1 (raw file):
useStopStateChangeConfirmation<StopBasicDetailsFormState>({ currentStopState: stop.quay?.stopState, stopPlaceRef: stop.stop_place_ref ?? '',
stopPlaceRef = quay.netex_id. Tässä pitäs mielellään käyttää suoraan quay:n netexId:tä. Tästä on kanssa ollut puhetta ja vähän työstä, että tän koko pysäkki sivun ei pitäs riippua ollenkaan nuista ScheduledStopPointeista.
ui/src/components/stop-registry/stops/stop-details/basic-details/BasicDetailsStopFields.tsx line 41 at r1 (raw file):
const { observationDate } = useObservationDateQueryParam(); const effectiveState = getEffectiveStopState(
Sama, ei arvota mitään uusia arvoja päivään perustuen. Tila on se miksi se on quayn tieodoissa asetettu.
This change is