#105_kontaktformular #121

muli · 2022-08-20T16:05:54+02:00

muli commented

2022-08-20 16:05:54 +02:00

Das Kontaktformular tut seinen Dienst bereits auf der Spielewiese mit dem Testotobo und soll hiermit auf die Hauptwebseite losgelassen werden.

Bekanntermaßen kaputt sind aktuell Umlaute. Ich bin mir aktuell noch nicht sicher, auf welcher Seite das gefixt werden muss. Bei der gefühlten Dringlichkeit können wir damit erstmal leben.

Das Kontaktformular tut seinen Dienst bereits auf der Spielewiese mit dem Testotobo und soll hiermit auf die Hauptwebseite losgelassen werden. Bekanntermaßen kaputt sind aktuell Umlaute. Ich bin mir aktuell noch nicht sicher, auf welcher Seite das gefixt werden muss. Bei der gefühlten Dringlichkeit können wir damit erstmal leben.

muli added 18 commits 2022-08-20 16:05:55 +02:00

feat: Added dev server to project file. 436be5830b

feat: WIP Kontaktformular hinzugefügt. (#105 ) aeb1ae24fa

Die URL ist noch hardcoded. Rückmeldung für den User fehlt noch. Die Nachricht geht noch nirgends hin. Spamprotection ohne Captcha ist nur in Ansätzen zu erkennen.

chore: Updated .gitignore. bbc1bc2b61

fix: Fixed Syntax in manifest.json. 83d4694190

feat: Bot-Detection erweitert. (#105 ) 5a089b6a2a

feat: Arrow functions aufgeräumt und Fehlerbehandlung ergänzt. (#105 ) 2af9144fcb

feat: Fehlermeldungen anzeigen und Botdetection anhand von Bearbeitunszeit. (#105 ) 487f2268b6

feat: Switched time based bot protection from JS to PHP sessions. 88276c2e2e

feat: Improved layout of contact form submit button. 553bba8917

fix: Removed empty class definition. 9b9f2b95c5

fix: Duzen all the way. 4f7edc53a8

feat: Added subject to contact form. e53449352f

fix: Remove opposite highlight classes to just display the most recent one. fa9cf1bf37

feat: Now really send emails. f58ba3bbc1

fix: Not just pass it along but really use the subject ;) 360f90fc9d

fix: Changed user facing text to formal speech. e2fa3f75cc

fix: Added missing error message for missing subject. 92d2c37000

feat: Switched from hardcoded domain to relative path to improve portability.

continuous-integration/drone/pr Build encountered an error

Details

b65755de8d

muli added 3 commits 2022-08-21 14:27:26 +02:00

fix: Fix sanitation to not break Umlauts and use specific email filter. 4e7b22fde6

fix: Ensure \r\n for line breaks and properly encode subject for umlauts. 54482d90f6

Merge pull request '#122_umlaute_kontaktformular' (#123 ) from #122_umlaute_kontaktformular into #105_kontaktformular

continuous-integration/drone/pr Build is passing

Details

2edfdcc179

Reviewed-on: #123

muli commented

2022-08-21 14:33:03 +02:00

Die Fixes für Umlaute sind jetzt drin.

Die Fehlermeldung von Drone sagt mir so erstmal nichts und ich habe wie gesagt für das Kontkatformular das Repo einmal neu geklont und Lektor via Pip neu installiert. Damit läuft der Dev-Server bei mir Fehlerfrei.

Ein Deployment auf die Spielwiese habe ich noch nicht probiert, da ich die Änderungen füs Kontaktformular noch nicht auf den Spielwiese-Branch rückportiert habe.

Die Fixes für Umlaute sind jetzt drin. Die Fehlermeldung von Drone sagt mir so erstmal nichts und ich habe wie gesagt für das Kontkatformular das Repo einmal neu geklont und Lektor via Pip neu installiert. Damit läuft der Dev-Server bei mir Fehlerfrei. Ein Deployment auf die Spielwiese habe ich noch nicht probiert, da ich die Änderungen füs Kontaktformular noch nicht auf den Spielwiese-Branch rückportiert habe.

Brain commented

2022-08-22 12:12:52 +02:00

So, Drone tut wieder. Die Registry hatte kurzzeitig vergessen, welche Images sie hat.

Man kann sich überlegen, dieses eigene Lektor-Image durch ein Python-Image, in dem man Lektor installiert zu ersetzen. Erhöht wahrscheinlich leicht die Build-Zeit, aber baut entsprechend mit aktuellen Versionen. Kann ich im Zuge von #118 ein Auge drauf werfen.

So, Drone tut wieder. Die Registry hatte kurzzeitig vergessen, welche Images sie hat. Man kann sich überlegen, dieses eigene Lektor-Image durch ein Python-Image, in dem man Lektor installiert zu ersetzen. Erhöht wahrscheinlich leicht die Build-Zeit, aber baut entsprechend mit aktuellen Versionen. Kann ich im Zuge von #118 ein Auge drauf werfen.

muli commented

2022-08-22 22:34:41 +02:00

Ahh, langsam verstehe ich von was für Images du redest 😅