Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates fro Karlsruhe. Contact email changed to [email protected] #167

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

digital-codes
Copy link
Collaborator

No description provided.

Copy link
Member

@silberzwiebel silberzwiebel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vielen vielen Dank!

Bitte noch die Kleinigkeiten ändern, die ich in den Kommentaren geschrieben habe.

Wo findet man denn die ÜBersichtskarte?
Hier habe ich aus Zeitgründen überhaupt nicht in den Code geschaut... Eine kurze Doku wäre noch gut, damit man sieht, wo was passiert.

Ansonsten gerne mergen und danke fürs Vorantreiben!

[[params.klimawatch.kommunen]]
name = "Karlsruhe"
slug = "karlsruhe"
[[params.klimawatch.kommunen]]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier bitte nur einmal Karlsruhe drin haben (wird bei jedem Aufrufen des Skripts ohne nachzuschauen ergänzt ...)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich kann gerade nicht nachvollziehen, warum die you-draw-it-Dateien neu generiert wurden. Aber scheint ja zu passen. Habe nicht alle Seiten geprüft, ob die Grafiken noch gut aussehen. Bei Zeit, bitte machen: https://deploy-preview-167--klimawatch.netlify.app/

Sonst wird es schon passen 😇

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gleiche wie oben: Habe nicht nachvollziehen können, warum die Grafiken für alle Kommunen neu generiert wurden. Aber werden schon passend aussehen (habe ich nicht geprüft).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitte mindestens noch 1-2 Sätze oben als Kommentar, was dieses Skript genau tut.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants