Changeset e9128bc in Klonkt for src/views/pages/admin-site-edit.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-site-edit.ejs (modified) (2 diffs)

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

@@ -179 +179 @@
     </fieldset>
 
+    <%# Aliassen en verhuizen staan sinds 14-8 op /admin/migrate, bij elkaar met
+        exporteren, importeren en ophalen (Robin: alle migratie-opties op de
+        migrate-pagina). Ze hoorden hier ook niet thuis: dit scherm gaat over hoe
+        je site eruitziet, en een verhuizing is geen uiterlijk. Alleen de
+        wegwijzer blijft, want wie ze hier zocht moet ze kunnen vinden. %>
     <fieldset>
       <legend><%= t('asite.aliases') %></legend>
-      <p class="form-hint"><%= t('asite.aliases_hint') %></p>
-      <label>
-<%# typeof-guard on purpose: during a deploy an old route can render this new
-    template for one request (view cache fills on first hit). A missing local
-    must degrade to an empty field, not take the whole page down with a 500. %>
-        <textarea name="ap_aliases" rows="3" placeholder="@oud@mastodon.social&#10;https://andere-klonkt.example/ap/users/naam"><%= typeof apAliases !== 'undefined' ? apAliases : '' %></textarea>
-      </label>
+      <p class="form-hint"><%= t('asite.moved_to_migrate') %></p>
+      <p><a href="/admin/migrate" class="btn"><%= t('admin.b_migrate') %></a></p>
     </fieldset>
 
@@ -237 +237 @@
   </form>
 
-<% if (!isNew) { %>
-  <form method="post" action="/admin/sites/<%= site.slug %>/move" class="admin-form"
-        onsubmit="return confirm('<%= t('asite.move_confirm') %>');">
-    <fieldset>
-      <legend><%= t('asite.move') %></legend>
-      <p class="form-hint"><%= t('asite.move_hint') %></p>
-      <% if (typeof site.moved_to === 'string' && site.moved_to) { %>
-        <p class="form-hint"><strong><%= t('asite.moved_to') %>:</strong> <%= site.moved_to %></p>
-      <% } %>
-      <label>
-        <input type="text" name="move_target" placeholder="@nieuw@andere-server.example" required>
-      </label>
-      <button type="submit" class="btn btn-danger"><%= t('asite.move_btn') %></button>
-    </fieldset>
-  </form>
-<% } %>
 </div>