Changeset e9128bc in Klonkt for src/views/pages/admin-migrate.ejs

Timestamp:

08/14/2026 02:12:31 AM (4 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

fc664ce

Parents:

dc802b4

Message:

Alle migratie-opties bij elkaar op één pagina

Robin: ik wil alle migratie-opties op de migrate-pagina hebben, dus ook
fediverse alias en move.

Ze stonden op de site-bewerkpagina, tussen de kleuren, de feedinstellingen en
de custom head-html. Dat is de verkeerde plek in twee opzichten: het zijn stap
1 en de laatste stap van een verhuizing, en dat scherm gaat over hoe je site
eruitziet. Een verhuizing is geen uiterlijk.

De pagina vertelt nu het verhaal in de goede volgorde:

1 je oude account claimen (alsoKnownAs)

meenemen (zip)
ophalen bij je oude Klonkt
hierheen halen (zip)

4 de verhuizing aankondigen (Move)

Claimen bovenaan omdat de bron zonder die claim niets afgeeft en de Move
weigert met no_backreference. Aankondigen onderaan met een rode knop: het is de
enige stap op deze pagina die niet terug te draaien is.

De verhuisknop post naar de BESTAANDE /admin/sites/:slug/move. Een tweede
implementatie van een onomkeerbare actie is precies wat je niet wilt; die route
kreeg alleen een next mee zodat je terugkomt waar je vandaan kwam.

EEN VAL DIE IK ONDERWEG MAAKTE. De opslagroute van het site-bewerkscherm
schreef ap_aliases uit het formulier. Haal je dat veld weg, dan komt er niets
binnen, en dan wist een gewone opslag je claim op je oude account. Je verandert
je accentkleur en drie stappen later weigert de Move met no_backreference,
zonder dat iets die twee met elkaar verbindt. Nu geldt: veld niet meegestuurd
is niet aanraken.

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

• POST /admin/migrate/aliases, met de bestaande parseApAliases
• aliassen en moved_to in de locals van alle drie de renders

src/routes/admin-sites.js

• parseApAliases geexporteerd in plaats van gekopieerd
• de save raakt ap_aliases alleen aan als het veld er echt is
• /move mag terugsturen naar een eigen pad (geen open redirect)

src/views/pages/admin-migrate.ejs

• alias bovenaan, verhuizen onderaan met een gewaarschuwde rode knop

src/views/pages/admin-site-edit.ejs

• beide blokken eruit, een wegwijzer ervoor in de plaats

src/services/i18n.js

• mig.alias_* en mig.move_* plus asite.moved_to_migrate, in nl/en/de

New file:
test/migrate-page.test.js

• de velden staan op de goede pagina en niet meer op de oude
• de volgorde: claimen boven ophalen, aankondigen onderaan
• en een alias overleeft het opslaan van je uiterlijk

remarks: gedraaid op een echte instantie. Alias opslaan werkt, een onzin-adres
geeft een nette fout EN laat de bestaande alias staan, en de pagina rendert
schoon in nl/en/de. Suite 1005 groen.

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

File:

• src/views/pages/admin-migrate.ejs (modified) (3 diffs)

src/views/pages/admin-migrate.ejs

@@ -5 +5 @@
   <% if (typeof success !== 'undefined' && success) { %><div class="alert alert-success"><%= success %></div><% } %>
   <% if (typeof error !== 'undefined' && error) { %><div class="alert alert-error"><%= error %></div><% } %>
+
+  <%# ── Je oude account claimen (FEP-7628 alsoKnownAs) ─────────── %>
+  <%# Bewust als EERSTE: zonder deze claim weigert de oude instantie de Move
+      met no_backreference, en kan de ophaalknop hieronder ook niets. Dit stond
+      op de site-bewerkpagina tussen de kleuren; daar zoekt niemand het als hij
+      aan het verhuizen is. %>
+  <section class="mig-card">
+    <h2><%= t('mig.alias_title') %></h2>
+    <p class="form-hint"><%= t('mig.alias_hint') %></p>
+    <form method="post" action="/admin/migrate/aliases">
+      <label for="mig-alias"><%= t('mig.alias_label') %></label>
+      <textarea id="mig-alias" name="ap_aliases" rows="3" class="mig-input"
+        placeholder="@oud@mastodon.social&#10;https://oude-klonkt.nl/ap/users/jij"><%= aliassen || '' %></textarea>
+      <p class="form-hint"><%= t('mig.alias_note') %></p>
+      <button type="submit" class="btn"><%= t('mig.alias_btn') %></button>
+    </form>
+  </section>
 
   <%# ── Exporteren ─────────────────────────────────────────────── %>
@@ -182 +199 @@
   </section>
 
+  <%# ── De verhuizing aankondigen (FEP-7628 Move) ──────────────── %>
+  <%# ONDERAAN, en met een rode knop. Dit is de enige knop op deze pagina die
+      de fediverse in gaat en niet terug te draaien is: je volgers verhuizen
+      mee en dit account gaat op slot. Alles hierboven kun je overdoen. %>
+  <section class="mig-card mig-card-danger">
+    <h2><%= t('mig.move_title') %></h2>
+    <p class="form-hint"><%= t('mig.move_hint') %></p>
+    <% if (movedTo) { %>
+      <p class="alert alert-error"><%= t('mig.move_done') %> <code><%= movedTo %></code></p>
+    <% } else { %>
+      <form method="post" action="/admin/sites/<%= site.slug %>/move"
+            onsubmit="return confirm('<%= t('mig.move_confirm') %>');">
+        <input type="hidden" name="next" value="/admin/migrate">
+        <label for="mig-move"><%= t('mig.move_label') %></label>
+        <input type="text" id="mig-move" name="move_target" class="mig-input"
+               placeholder="@nieuw@andere-server.nl" required>
+        <p class="form-hint mig-warn"><%= t('mig.move_warn') %></p>
+        <button type="submit" class="btn btn-danger"><%= t('mig.move_btn') %></button>
+      </form>
+    <% } %>
+  </section>
+
   <%# ── Wie je volgt ────────────────────────────────────────────── %>
   <section class="mig-card">
@@ -206 +245 @@
   .mig-report code { word-break: break-all; font-size: .88rem; }
   .mig-warn { color: var(--accent); }
+  .mig-card-danger { border-color: color-mix(in srgb, var(--accent, #c33) 45%, var(--rule)); }
+  textarea.mig-input { font-family: inherit; resize: vertical; }
   .mig-report { border-left: 3px solid var(--accent); padding: .8rem 1rem; margin-bottom: 1.2rem;
                 background: color-mix(in srgb, var(--ink, #000) 4%, transparent); border-radius: 0 8px 8px 0; }