Changeset 47f1183 in Klonkt for src/routes/admin-migrate.js

Timestamp:

08/14/2026 03:02:35 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

7e474f7

Parents:

f704237

Message:

Een stappenplan op de migrate-pagina, en de volgorde klopte niet

Robin: misschien moeten we bij migrate een stappenplan beschrijven, waaronder
dus ook dit (de Delete-ronde voor je een domein opzegt).

Bij het uitschrijven bleek de pagina zelf een verkeerde volgorde te suggereren.
Ik noemde het aankondigen "de laatste stap", maar ingestFromSource weigert met
not_moved_here zolang movedTo niet naar ons wijst. De ophaalknop werkt dus PAS
nadat de Move gedaan is. Aankondigen is stap 2, niet de afsluiter.

Met een zip mag het juist andersom: een bestand vraagt niemand toestemming. Die
twee volgordes naast elkaar zijn precies het soort ding waar je halverwege op
vastloopt zonder te snappen waarom, dus staan ze er nu uitgeschreven.

DE PAGINA DRAAIT OP BEIDE INSTANTIES. Stap 1, 3 en 4 horen op de nieuwe kant,
stap 2 en 5 op de oude. Een vast "hier" is dus op de helft van de schermen
onzin. Het plan draait mee met waar je kijkt, en op de vertrekkende kant staat
er een regel boven dat je naar de instantie kijkt die weggaat.

Die omkering had ik bij het bouwen precies verkeerd om: beide labels werden
"hier". Zag ik pas door hem te renderen, niet door hem te lezen.

STAP 5 heeft geen knop, en dat staat erbij. Verwijder je oude berichten voor je
het domein opzegt, zodat boosts van anderen netjes verdwijnen in plaats van te
blijven staan als kapotte kaart; haal daarna de alias weg, want die wijst naar
een adres dat je niet meer beheert. Een stappenplan dat een knop belooft die er
niet is, is erger dan geen stappenplan.

Changed files:
src/routes/admin-migrate.js

• stappen(): afleiden wat af is uit aliassen, moved_to, de moves-collectie en de vertaaltabel; meegegeven aan alle drie de renders

src/views/pages/admin-migrate.ejs

• het stappenplan bovenaan, met vinkjes en een waar-label per stap
• de labels draaien om vanaf de vertrekkende kant

src/services/i18n.js

• mig.plan_* in nl/en/de
• mig.move_title is "Stap 2" en niet meer "Laatste stap"
• de waarschuwing zegt nu dat dit EERST moet bij de ophaalknop

test/migrate-page.test.js

• het plan staat boven de knoppen die het beschrijft
• de perspectiefwissel, met de eis dat de twee labels nooit gelijk zijn
• geen "laatste stap" meer in een van de drie talen
• stap 5 noemt de Delete-ronde, de alias en dat er geen knop is

remarks: gerenderd vanaf beide kanten om het te controleren. Suite 1013 groen,
tegenproef op de perspectiefwissel.

-robo
Co-Authored-By: Claude Opus 4.8 <noreply@…>

File:

• src/routes/admin-migrate.js (modified) (5 diffs)

src/routes/admin-migrate.js

@@ -80 +80 @@
 
 /**
+ * Waar sta je in de verhuizing?
+ *
+ * Deze pagina draait op BEIDE instanties en elke stap hoort maar op een van de
+ * twee. Zonder dat onderscheid leest de lijst als onzin op de helft van de
+ * schermen. Vandaar: afleiden wat we kunnen zien, en verder eerlijk zeggen
+ * waar iets thuishoort in plaats van het te raden.
+ */
+function stappen(site, mig) {
+  const alias = aliasTekst(site).trim();
+  return {
+    // moved_to gezet betekent: DIT is de instantie die vertrokken is.
+    isOud: !!(site && site.moved_to),
+    geclaimd: !!alias,
+    // Een Move in onze moves-collectie betekent dat de bron hierheen verhuisd is.
+    aangekondigd: !!(site && site.moved_to) || !!(mig && mig.moves > 0),
+    opgehaald: !!(mig && mig.total > 0),
+  };
+}
+
+/**
  * Waar zouden we vandaan kunnen halen? De alias die we zelf claimen (FEP-7628
  * alsoKnownAs). Dat is niet toevallig hetzelfde veld als waar de ingest op
@@ -97 +117 @@
   const site = res.locals.site;
   const { telling, fout } = tellen(site);
+  const _mig = site ? Migration.migrationStatus(site.slug) : null;
   renderPage(req, res, 'pages/admin-migrate', {
     pageTitle: 'Migreren', bodyClass: 'on-special',
@@ -102 +123 @@
     verslag: null,
     bron: bronKandidaat(site), aliassen: aliasTekst(site), movedTo: (site && site.moved_to) || null,
-    mig: site ? Migration.migrationStatus(site.slug) : null,
+    mig: _mig, stap: stappen(site, _mig),
     haalVerslag: null,
     success: req.query.success || null, error: req.query.error || fout || null,
@@ -137 +158 @@
   }
   const { telling } = tellen(site);
+  const _mig2 = Migration.migrationStatus(site.slug);
   renderPage(req, res, 'pages/admin-migrate', {
     pageTitle: 'Migreren', bodyClass: 'on-special',
     telling, fout: null, mb, verslag: null,
     bron: opgegeven || bronKandidaat(site), aliassen: aliasTekst(site), movedTo: site.moved_to || null,
-    mig: Migration.migrationStatus(site.slug),
+    mig: _mig2, stap: stappen(site, _mig2),
     haalVerslag: r,
     success: (r && !r.error) ? 'Opgehaald' : null,
@@ -227 +249 @@
     telling: tellen(site).telling, fout: null, mb,
     bron: bronKandidaat(site), aliassen: aliasTekst(site), movedTo: site.moved_to || null,
-    mig: Migration.migrationStatus(site.slug), haalVerslag: null,
+    mig: Migration.migrationStatus(site.slug), stap: stappen(site, Migration.migrationStatus(site.slug)), haalVerslag: null,
     verslag: verslag ? { ...verslag, echt, overschrijf, bestand: req.file.originalname, bytes: req.file.buffer.length } : null,
     success: (echt && verslag && !fout) ? 'Archief geïmporteerd' : null,