search

StrategieLukas / stadt-land-klima

3 stars 1 forks source link

Added Flow to automatically set municipality type when population is … #115

Closed StrategieLukas closed 10 months ago

StrategieLukas commented 10 months ago

Added Flow to automatically set municipality type when population is set; Added "Mittelstadt" Implements #78

paul-severin commented 10 months ago

oder so, mit besser developer experience, testbarkeit etc 🙃 : https://github.com/LukasLanger/stadt-land-klima/pull/116/files

theodorun commented 10 months ago

@LukasLange pls apply the changes than, I can merge

StrategieLukas commented 10 months ago

@LukasLanger pls apply the changes than, I can merge

Glaub den PR hier können wir eigentlich schließen und stattdessen den von Paul nehmen ( https://github.com/LukasLanger/stadt-land-klima/pull/116 ), er hat ja auf dem selben Branch weitergearbeitet Ich hab aber das Gefühl, bei dem PR 116 von @paul-severin fehlen ein paar Sachen, also die 2 .yamls hier, welche das schema im UI verändern und die Validierung für die Großstadtgröße müssten in dem PR am besten noch aufgenommen werden, dann kann der hier geschlossen werden

paul-severin commented 10 months ago

habe die fehlenden yamls in mein PR übernommen: https://github.com/LukasLanger/stadt-land-klima/pull/116#commits-pushed-77ffb16

StrategieLukas commented 10 months ago

habe die fehlenden yamls in mein PR übernommen: #116 (commits)

und auch schon die review comments von ondrej, wie ich sehe - sehr nice, dann schließ ich den PR hier mal :)