Skip to content

Add chaoszone entry - #56

Merged
teuserer merged 2 commits into
netz39:mainfrom
Fuxmoder:main
Oct 14, 2026
Merged

teuserer merged 2 commits into
netz39:mainfrom
Fuxmoder:main

Conversation

@Fuxmoder

@Fuxmoder Fuxmoder commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

makes chaoszone services able to find the api automatically
this would display our contact data and open/close condition on chaoszone.cz

makes chaoszone services able to find the api automatically
LeSpocky
LeSpocky previously approved these changes Oct 12, 2026

@penguineer penguineer left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

From a technical perspective that PR looks sound.

I do not understand what the change will affect. Please describe in the PR and, ideally, in a short comment above the entry; especially since it is a custom extension and we cannot look it up in an API doc.

makes chaoszone services able to find the api automatically
displays our contact data and open/closed status on chaoszone.cz
Comment thread src/SpaceApiEntry.py

@teuserer teuserer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wir sind offiziell Teil der ChaosZone. Ich habe da keine Enwände, dass diese Änderung an sich einen Unterschied macht. Gerne morgen nach dem Plenum mergen.

@penguineer

Copy link
Copy Markdown
Member

Wir sind offiziell Teil der ChaosZone.

Solche Infos gern direkt mit in die PR-Beschreibung, sonst müssen die Maintainer das prüfen und haben Extra-Aufwand.

@Fuxmoder

Copy link
Copy Markdown
Contributor Author

Wir sind offiziell Teil der ChaosZone.

Solche Infos gern direkt mit in die PR-Beschreibung, sonst müssen die Maintainer das prüfen und haben Extra-Aufwand.

Es gab bei dem Thema einfach zu viel missverständnisse und es wurde einfach zu wenig dokumentiert
Werde in zukunft besser drauf achten das die PRs informativer sind und tut mir leid für die Umstände, war leider auch mein erstes mal XD

@penguineer

Copy link
Copy Markdown
Member

Es gab bei dem Thema einfach zu viel missverständnisse und es wurde einfach zu wenig dokumentiert Werde in zukunft besser drauf achten das die PRs informativer sind und tut mir leid für die Umstände, war leider auch mein erstes mal XD

Danke! Meine Anmerkung geht tatsächlich auch an die gesamte Community, weil ich mich an Diskussionen erinnere, aber selbst auch kein Ergebnis gefunden habe. Dieser PR soll ja eine Lücke schließen, aber ich konnte diese Lücke nicht nachvollziehen.

Letztendlich ist es auch kein Riesending, die Ergänzung aufzunehmen. Ich habe die Diskussion aufgemacht, weil sich diese Lücken gezeigt haben. Die Verantwortung liegt auch bei denen, die schon länger dabei sind, etwas zu hinterlassen, mit dem neue Mitglieder weiter machen können.

@Fuxmoder

Copy link
Copy Markdown
Contributor Author

Hab es die letzten Monaten leider selber auf unangenehme Weise merken dürfen. Aber das heisst ja nicht das wir das nicht besser hinkriegen können. Auch so bisschen Errinnern, das wer etwas übernimmt auch die dokumentation übernehmen muss und wenn was fehlt darauf hinweisen.

Und jetzt liegt einfach vieles in den Stammtischen drin, und wenn ich jetzt bspw. anfange das thema regelmässig kurz anzureissen (stichwort telko oder project updates) müsstest du da plötzlich 50 Mitschriften durchlesen um zum Punkt zu kommen.

@teuserer
teuserer merged commit 23aa30c into netz39:main Oct 14, 2026
2 checks passed
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.

4 participants