Updates to terminal and stop change histories

[suvikankare]

• Extract common code for external links
• Refactor external links diffing ui
• Implement diffing external links in stops

---

This change is 

[github-actions]

Dependency Review

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

Scanned Files

None

[PasiVuohijoki]

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

---

ui/src/components/stop-registry/stops/change-history/utils/dataDiffers/diffExternalLinks.tsx line 19 at r1 (raw file):

  current: HistoricalStopData,
): Array<ChangedValue> {
  const formatLinks = (links: readonly (ExternalLink | null)[]) => {

Nit: Kun tuo ExternalLinksList ja formatExternalLinks on tuolla samassa moduulissa, niin tää funktiokin vois olla siellä. t-funktio sitte vaan parametrina menemään.

---

ui/src/components/stop-registry/stops/change-history/utils/dataDiffers/diffExternalLinks.tsx line 20 at r1 (raw file):

): Array<ChangedValue> {
  const formatLinks = (links: readonly (ExternalLink | null)[]) => {
    const formatted = formatExternalLinks(compact(links ?? []));

Nopeesti vilkaistuna tää compactin käyttö on tässä turha, kun tuo formatExternalLinks sisäisesti käyttää tota compact(links) asiaa.

---

ui/src/components/stop-registry/terminals/change-history/utils/diffTerminal.tsx line 308 at r1 (raw file):

): Array<ChangedValue> {
  const formatLinks = (links: readonly (ExternalLink | null)[]) => {
    const formatted = formatExternalLinks(compact(links ?? []));

Tässä sama homma ku aiemmin mainitsin. Compactin voi jättää veke.

[suvikankare]

@suvikankare made 2 comments and resolved 1 discussion.
Reviewable status: 0 of 13 files reviewed, 2 unresolved discussions (waiting on PasiVuohijoki).

---

ui/src/components/stop-registry/stops/change-history/utils/dataDiffers/diffExternalLinks.tsx line 20 at r1 (raw file):

Previously, PasiVuohijoki (Pasi Vuohijoki) wrote…

Nopeesti vilkaistuna tää compactin käyttö on tässä turha, kun tuo formatExternalLinks sisäisesti käyttää tota compact(links) asiaa.

Done.

---

ui/src/components/stop-registry/terminals/change-history/utils/diffTerminal.tsx line 308 at r1 (raw file):

Previously, PasiVuohijoki (Pasi Vuohijoki) wrote…

Tässä sama homma ku aiemmin mainitsin. Compactin voi jättää veke.

Done.

[PasiVuohijoki]

@PasiVuohijoki reviewed 13 files and all commit messages, and resolved 2 discussions.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on suvikankare).