DRAFT: Stop change history latest

[suvikankare]

This change is 

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[Huulivoide]

@Huulivoide reviewed all commit messages and made 3 comments.
Reviewable status: 0 of 16 files reviewed, 3 unresolved discussions (waiting on suvikankare).

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 49 at r1 (raw file):

          {historyItem.privateCodeType === 'HSL/JORE-3'
            ? t('stopChangeHistory.importedVersion')
            : t('stopChangeHistory.firstVersion')}

Voisi olla myös kopio. ui/src/components/stop-registry/stops/change-history/components/NoPreviousChangeVersionSection.tsx

const copiedVersionsImportedIdRegexp = /HSL:Quay:\d+-\d{4}-\d{2}-\d{2}-\d+/;
function determineType(historyItem: QuayChangeHistoryItem): ChangeVersionType {
  if (historyItem.importedId?.match(copiedVersionsImportedIdRegexp)) {
    return 'copied';
  }

  if (historyItem.privateCodeType === 'HSL/JORE-3') {
    return 'imported';
  }

  return 'created';
}

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryTable.tsx line 125 at r1 (raw file):

  const { historyItems, loading, error, refetch } =
    useGetStopChangeHistoryItems({

Lähtökohtasestihan Quayn ei pitäs päivittyä, jos sitä ei oo oikeesti mitnekään muutettu. Täten, jos noutaa vaan 5 viimesintä, versiota, niin pitäs näkyviin 5 viiimisintä muutosta. Eli vois olla joku oma kysely joka nakkaa oikeen sortin paikoilleen, ja sit limit: 5

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryTable.tsx line 135 at r1 (raw file):

    });

  const { showLoader, previousHistoryItems } = usePrettyLoaderState(

Tää oli tuolla muutossivulla, sen takia, että ku siellä voi muuttaa sitä järjestystä ja vaihtaa aika suodatinta nuolinapeilla päivä kerrallaan, niin vaikka sieltä kannasta tulikin vastaukset parissakymmenessä millisekunnissa, niin UI:lla se tila ehti kuitenkin vaihtuu ladatun ja latauksen ja spinneri välähti ruudulla yhden framin ajan.

Täällä ton sisällä ei pitäs koskaan olla tommosta tilannetta.

[suvikankare]

@suvikankare made 3 comments.
Reviewable status: 0 of 20 files reviewed, 3 unresolved discussions (waiting on Huulivoide).

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 49 at r1 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Voisi olla myös kopio. ui/src/components/stop-registry/stops/change-history/components/NoPreviousChangeVersionSection.tsx

const copiedVersionsImportedIdRegexp = /HSL:Quay:\d+-\d{4}-\d{2}-\d{2}-\d+/;
function determineType(historyItem: QuayChangeHistoryItem): ChangeVersionType {
  if (historyItem.importedId?.match(copiedVersionsImportedIdRegexp)) {
    return 'copied';
  }

  if (historyItem.privateCodeType === 'HSL/JORE-3') {
    return 'imported';
  }

  return 'created';
}

Done.

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryTable.tsx line 125 at r1 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Lähtökohtasestihan Quayn ei pitäs päivittyä, jos sitä ei oo oikeesti mitnekään muutettu. Täten, jos noutaa vaan 5 viimesintä, versiota, niin pitäs näkyviin 5 viiimisintä muutosta. Eli vois olla joku oma kysely joka nakkaa oikeen sortin paikoilleen, ja sit limit: 5

limit: 6 koska muuten näyttää aina viimeisimpänä sen imported, muuten done

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryTable.tsx line 135 at r1 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Tää oli tuolla muutossivulla, sen takia, että ku siellä voi muuttaa sitä järjestystä ja vaihtaa aika suodatinta nuolinapeilla päivä kerrallaan, niin vaikka sieltä kannasta tulikin vastaukset parissakymmenessä millisekunnissa, niin UI:lla se tila ehti kuitenkin vaihtuu ladatun ja latauksen ja spinneri välähti ruudulla yhden framin ajan.

Täällä ton sisällä ei pitäs koskaan olla tommosta tilannetta.

Poistettu

[Huulivoide]

@Huulivoide reviewed 20 files and all commit messages, made 2 comments, and resolved 3 discussions.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on suvikankare).

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 51 at r3 (raw file):

      <div className="mb-3 text-sm font-semibold">
        <Link
          to={routeDetails[Path.stopChangeHistory].getLink(publicCode)}

Alempana tarkempi kommentit, mutta prioriteetti mukaan

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 119 at r3 (raw file):

        <div key={section.title}>
          <Link
            to={routeDetails[Path.stopChangeHistory].getLink(publicCode)}

prioriteetti pitäs olla mukana kanssa, ja sit ehkä vois olla aiheellista varmistaa että täältä tulee from ja to datet paikoilleen siten että siel ruudul sit näkyis se haluttu muutos. Tosin sit kyl pitös varmaan arpoo myös oikee pakinointi sivu paikoilleen, ja ois varmaan kiva jos se aut skrollais oikeeseen kohtaan vielä 🙈

Prioriteetti mukaan, se on tärkein, muut voi olla jatkokehitystä.

[Huulivoide]

Sit vielä tuli mieleen, tälleen ei blokkaavana kommenttina että voisit miettii:

1. Oisko järkee splitata noita komponentteja pienempiin palasiin, Loader, Error, View state omiin ali palasiinsa yms.
2. Miettii jo etukäteen vähän Pysäkkialuuen ja Terminaalin vastaavia komponentteja, ja että onko tässä mitään semmosta koodin pätkää, jonka vois jo valmiiks kirjottaa jotenkin geneerisemmässä muodossa (jos ei jo ole).

@Huulivoide reviewed 3 files and all commit messages, made 1 comment, and resolved 2 discussions.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on suvikankare).

[Huulivoide]

@Huulivoide reviewed 8 files and all commit messages, and made 3 comments.
Reviewable status: 25 of 27 files reviewed, 3 unresolved discussions (waiting on suvikankare).

---

ui/src/components/common/ChangeHistory/LatestChangeHistoryItem.tsx line 20 at r5 (raw file):

  };
  readonly sections: ReadonlyArray<ChangeSection>;
  readonly link: string;

Teknisesti tyyppi vois olla To

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 60 at r5 (raw file):

    previousCached?.status !== 'fetched'
  ) {
    return <LoadingState />;

Pitäs varmaan olla FailedToLoadState

---

ui/src/components/stop-registry/stops/change-history/utils/latestStopChangeSections.ts line 19 at r5 (raw file):

};

export const latestStopChangeSections = (

ylätasolla niin function latest...

[Huulivoide]

@Huulivoide made 1 comment.
Reviewable status: 25 of 27 files reviewed, 4 unresolved discussions (waiting on suvikankare).

---

ui/src/components/common/ChangeHistory/ErrorLoadingState.tsx line 10 at r5 (raw file):

};

export const ErrorLoadingState: FC<ErrorLoadingStateProps> = ({

Täällä vois hartkita alikansiota tai laittaa näihin load state komponentteihin sen LatestChangeHistory prefiksin mukaan, koska nää liittyy siihen ominaisuuteen, eikä nää oo käytössä muutoshistoriasivun puolella.

[suvikankare]

@suvikankare made 4 comments.
Reviewable status: 17 of 28 files reviewed, 4 unresolved discussions (waiting on Huulivoide).

---

ui/src/components/stop-registry/stops/change-history/components/LatestStopChangeHistoryItem.tsx line 60 at r5 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Pitäs varmaan olla FailedToLoadState

Done.

---

ui/src/components/stop-registry/stops/change-history/utils/latestStopChangeSections.ts line 19 at r5 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

ylätasolla niin function latest...

Done.

---

ui/src/components/common/ChangeHistory/ErrorLoadingState.tsx line 10 at r5 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Täällä vois hartkita alikansiota tai laittaa näihin load state komponentteihin sen LatestChangeHistory prefiksin mukaan, koska nää liittyy siihen ominaisuuteen, eikä nää oo käytössä muutoshistoriasivun puolella.

Done.

---

ui/src/components/common/ChangeHistory/LatestChangeHistoryItem.tsx line 20 at r5 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Teknisesti tyyppi vois olla To

Done.

[Huulivoide]

@Huulivoide partially reviewed 11 files and all commit messages, and resolved 4 discussions.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on suvikankare).