FriendsOfREDAXO / maintenance

Wartungsmodus: Sperrung des Frontends für Besucher und des Backends für Redakteure
MIT License
52 stars 7 forks source link

Überarbeitung des Maintenance-Addons #81

Closed alxndr-w closed 4 months ago

alxndr-w commented 4 months ago

closes #77, closes #78

Neue Funktionen

Aktuell entfernt

Außerdem zu beachten

skerbis commented 4 months ago

@eaCe Automatische Tests , kannst du hier dann noch mal behilflich sein?

alxndr-w commented 4 months ago

@eaCe @skerbis aber erst, wenn ich soweit bin...

alxndr-w commented 4 months ago

Ich würde hier Mal kurz "Pause" machen, die Funktionen sind alle noch nicht Mal von mir stichprobenhaft getestet, sondern einfach nur ein Feature nach der anderen neu programmiert.

Wie eingangs erwähnt habe ich auch die automatischen Tests ausbauen müssen, weil "nicht darum gekümmert" und "keine Ahnung, wie das geht".

Mein Wunsch-Vorgehen ist: Alle Einstellungen testen (auch in Kombination, wo sinnvoll), dann erst wieder automatische Tests einbauen und das Tagging-Skript für die IP- und Hosteingabe integrieren.

skerbis commented 4 months ago

Ich erhalte bei Sperrung

Call to undefined method rex_addon::isIpAllowed() Datei: redaxo/src/addons/maintenance/lib/Maintenance.php:122

alxndr-w commented 4 months ago

Alles bis auf die automatischen Tests wurde jetzt erledigt.

Fehlende Rückwärtskompatibilität an folgenden Stellen:

  1. Fragment-Pfade haben sich geändert
  2. Parameternamen der Config haben sich geändert
  3. Keine Tests. Das wäre meine Bitte an @eace, der sich bisher nicht zu Wort gemeldet hat.

Ich könnte damit leben, den PR auch so anzunehmen - dann allerdings die Version als 3.0.0-beta kennzeichnen und in der Community zum Prüfen aufrufen.

eaCe commented 4 months ago

Die Tests laufen gar nicht in deinen Commits? Ich muss mir den PR mal runterladen und die Tests prüfen. Vielleicht komme ich in dieser Woche dazu. Am Wochenende ist leider alles voll.