search

match4everyone / match4healthcare

Helping hospitals find qualified medicine students (Hackathon #wirvsvirus). A more flexibel version is currently in development under match4everyone/match4everything,
https://match4healthcare.de
Other
17 stars 10 forks source link

Add sample gateway error page #538

Closed maltezacharias closed 4 years ago

maltezacharias commented 4 years ago

Adds a couple of files to the repository that show how to setup an error page for when the app server is not running on an nginx gateway. Also includes stripped down bootstrap layout and a sticky footer without JS which could be interesting for m4e

maltezacharias commented 4 years ago

@bjrne Ich krieg da nen Fehler weil ich ein File >500kb versuche zu comitten. Ich glaub die Grenze ist zu niedrig. Ich hab das Bild so klein gemacht wie es geht, aber wenn man auch 4K Displays bedienen will geht einfach nicht weniger

bjrne commented 4 years ago

Ich hab da keine Grenze festgelegt, das ist ein sensible default. Ich finde es ist lange an der Zeit, lfs für sowas zu benutzen. Bilder gehören einfach nicht ins git, es ist ja auch mega nervig wenn man ein repo frisch (grade unterwegs) und er erstmal 20MB an Dateien aus der History runterlädt, die man garnicht mehr braucht.

Also: lets setup lfs :tada:, kann das auch gern machen :)

maltezacharias commented 4 years ago

Ich musste LFS jetzt erstmal googlen, für ne Handvoll Bilder bin ich dagegen. Und 20MB oder selbst 200MB sind halt auch nicht wirklich viel.. Irgendwann wird Deutschland sicher auch noch ein funktionierendes Mobilfunknetz bekommen..

bjrne commented 4 years ago

Wenn man das noch nie gesehen hat, wirkt es wie overkill. In der Praxis merkst du aber den Unterschied nur durch den schnelleren checkout und musst manuell garnichts machen. Es macht technisch halt keinen Sinn, Bilder zeilenweise ins .git zu speichern. Der Speicherplatz akkumuliert sich ja im git durch die history, die du mit auscheckst.

bjrne commented 4 years ago

Durch das sinnfreie 1GB pull-bandwith limit von github hat sich lfs vllt erledigt. Leider dauert der checkout schon jetzt einige Zeit, weil das runterladen von binary files über das git interface nicht besonders optimiert ist.

maltezacharias commented 4 years ago

Ahh perfekt, ich hatte sowas gelesen aber keine aktuelle Doku zu den Terms & Conditions gefunden. Klares Kontra LFS in dem Fall. Mich hat auch gestört, dass es dadurch schwieriger wird einfach einen lokalen Git-Server zu haben. Gibt's inhaltlich zum PR Fragen oder können wir mergen?

maltezacharias commented 4 years ago

@kevihiiin integrated your changes, as for testing, this is a sample file not meant to be runnable without some further tinkering. It was pulled from the prod system though, so we've been testing it for a while