search

lebenswurzel / solawi-bedarf

Eine App für die Produktverwaltung und Bedarfsanmeldung in einer SoLawi
GNU Affero General Public License v3.0
2 stars 0 forks source link

Change default PDF table layout #73

Closed R1tschY closed 2 months ago

R1tschY commented 2 months ago

Fixes #39

joergrs commented 2 months ago

Habe nicht jede Änderung bis ins Detail nachvollzogen, aber finde dein Refactoring trotzdem schon super. Die PDFs sehen jetzt deutlich aufgeräumter aus und gut ist auch, dass für die Einheit keine separate Spalte mehr verbraucht wird.

Würdest du guten Gewissens sagen, dass wir es nach main übernehmen können? Wenn wir es morgen schon Releasen, könnten die Mitarbeiter schon mal ausprobieren, wie sie mit dem neuen PDF-Format klar kommen und uns ggf. vor der kommenden Verteilung Rückmeldung geben, ob es soweit passt.

Habe den Branch auf den aktuellen main rebased. #37 ist ja hier auch mit drin, deswegen sollten wir #67 auf jeden Fall verwerfen (mache ich mal gerade).

R1tschY commented 2 months ago

Du kannst es gerne nochmal ausprobieren und die Commits reviewen die ich noch lokal rumliegen hatte. Aber ich denke, man kann es releasen. Heute Abend habe ich aber keine Zeit nochmal rein zu schauen.