FachschaftMathPhysInfo / Finance-Visualizer

Gibt eine Übersicht über die Finanzen der Fachschaft MathPhysInfo
http://www.mathphys.info/finanzen/
GNU General Public License v3.0
4 stars 2 forks source link

Unterstützung für mehrere Jahre in ein- und derselben Instanz #11

Closed TuringTux closed 4 years ago

TuringTux commented 4 years ago

Closes #3.

Ich habe mich mal daran versucht, die Unterstützung für mehrere Jahre zu implementieren. Ich drifte leider insbesondere bei großen Änderungen gerne ab und verliere den Überblick, wenn ich mir keine klare Struktur für die Anwendung vornehme. Bei diesem Projekt habe ich versucht, alle Änderungen in eine klare Struktur einzubauen.

Dementsprechend viele geänderte Dateien enthält dieser Pull Request, wofür ich jetzt schon um Entschuldigung bitten möchte. Ich hoffe, alles ist einigermaßen verständlich, falls nicht, bitte schreibt mir, dann verbessere ich die Dokumentation.

Ich hoffe auch, ich habe den Code nicht zu sehr massakriert. Falls doch, schreibt mir, dann mache ich die gewünschten Änderungen :smiley:

Aktuell noch WIP, weil noch nicht alles fertig ist.

Änderungen am UI

Änderungen am Datenformat

:warning: Ich habe die Daten nach bestem Wissen und Gewissen übertragen, aber vermutlich wäre es gut, jemand schaut noch mal drüber, bevor die Änderungen live gehen.

Änderungen an der Anwendungsstruktur

TuringTux commented 4 years ago

@ThorstenBuss An zwei Stellen habe ich einen // Bit of an ugly hack, hopefully that's okay...-Kommentar, die könnte ich noch überarbeiten, wenn gewünscht.

Ansonsten bin ich von meiner Seite aus fertig.

TuringTux commented 4 years ago

Hey, vielen Dank für das Review und das positive Feedback! :blush:

Ich habe den Year-Switcher verschoben, und das README angepasst. Ich habe zur Entwicklungszeit immer Pythons Http-Server verwendet, aber du kannst sehr gerne in die Liste noch andere Vorschläge reinschreiben (ich kann das auch machen, aber wenn du das committen könntest, wäre das super (damit halt die Autorenschaft richtig zugeordnet ist)).

ThorstenBuss commented 4 years ago

Ich muss mich Tom anschließen, ich finde den pull request auch super. Die Stellen mit den // Bit of an ugly hack-Kommentaren könnte man vielleicht dadurch verschönern, dass man den Nahmen der Fachschaft und die URL zur Website einmalig an einer Zentralen Stelle definiert.

TuringTux commented 4 years ago

Dankeschön für das nette Feedback!

Ja, das wäre eine gute Idee zur Verschönerung. Ich schau mal noch, wie man das am besten umsetzt. Wir Computerlinguisten haben ja auch eine Instanz des Finanz-Tools laufen, das heißt, ich würde dann gleich mal darauf achten, die Konfigurationen auf einen gemeinsamen Nenner zu bringen, dass an unserem Fork möglichst wenig geändert werden muss.