Comenius-Institut/FOERBICO
Comenius-Institut/FOERBICO
14
5
Fork 2
Code Issues124 Pull requests9 Projects6 Releases Wiki Activity

Korrekturen für den Artikel "OER meets Fachdidaktik" #300

Merged
sicking merged 5 commits from bugfix/correct_blogartikel-oer-meets-fachdidaktik-aka-KLT into main 2025-03-11 08:53:03 +00:00
Conversation 9 Commits 5 Files changed 5 +10 -8
sicking commented 2025-03-10 13:48:47 +00:00
Owner
  • der Blogpost-Ordner " 2025-02-25-KLT-Münster" enthielt ein führendes Leerzeichen
  • die (technischen) Metadaten (für HUGO) fehlten (Abschnitt #staticSiteGenerator:: u. a. Autoren, Cover-Bild, url
  • Zeilenumbrüche ergänzt, Anmerkungen von "markdownlint"

Die von diesem Branch erstellte Webseite liegt zur Prüfung und Abnahme hier: https://int.oer.community/ ( - das "int." am Anfang macht den Unterschied - )

- der Blogpost-Ordner " 2025-02-25-KLT-Münster" enthielt ein führendes Leerzeichen - die (technischen) Metadaten (für HUGO) fehlten (Abschnitt `#staticSiteGenerator:`: u. a. Autoren, Cover-Bild, url - Zeilenumbrüche ergänzt, Anmerkungen von "markdownlint" Die von diesem Branch erstellte Webseite liegt zur Prüfung und Abnahme hier: https://int.oer.community/ ( - das "**int.**" am Anfang macht den Unterschied - )
sicking added 5 commits 2025-03-10 13:48:47 +00:00 
- Unterordner gelöscht
- Titel geändert, Bsp. für Auswirkung dieses (technischen) Metadatums
- Kleinigkeiten (Leerzeichen, Leerzeilen...)
sicking requested review from phillipangelina 2025-03-10 13:49:04 +00:00
sicking requested review from lmoessle 2025-03-10 13:49:15 +00:00
sicking requested review from joerglohrer 2025-03-10 13:49:20 +00:00
sicking commented 2025-03-10 13:51:10 +00:00
Author
Owner

Ein Unterschied zur aktuell veröffentlichten Seite ist z. B. der Link zur KTF Münster im Artikel "OER meets Fachdidaktik" unter dem Bild bei der Copyright-Angabe.

Kann das so raus?

Ein Unterschied zur aktuell veröffentlichten Seite ist z. B. der Link zur KTF Münster im [Artikel "OER meets Fachdidaktik"](https://int.oer.community/oer-meets-fachdidaktik/) unter dem Bild bei der Copyright-Angabe. Kann das so raus?
joerglohrer commented 2025-03-10 14:38:28 +00:00
Owner

Von mir aus passt das. Muss ich noch was tun und den Review zu bewilligen oder reicht der Kommentar hier?

Von mir aus passt das. Muss ich noch was tun und den Review zu bewilligen oder reicht der Kommentar hier?
joerglohrer closed this pull request 2025-03-10 14:38:28 +00:00
joerglohrer reopened this pull request 2025-03-10 14:39:14 +00:00
joerglohrer commented 2025-03-10 14:39:53 +00:00
Owner

Sorry hatte versehentlich geschlossen und will den Reviews von @phillipangelina und @lmoessle nicht vorgreifen

Sorry hatte versehentlich geschlossen und will den Reviews von @phillipangelina und @lmoessle nicht vorgreifen
lmoessle commented 2025-03-10 15:35:45 +00:00
Member

Passt von meiner Seite aus auch! Denke für @phillipangelina auch

Passt von meiner Seite aus auch! Denke für @phillipangelina auch
phillipangelina commented 2025-03-10 16:13:07 +00:00
Member

Ja von meiner Seite auch aus

Ja von meiner Seite auch aus
sicking commented 2025-03-11 08:51:12 +00:00
Author
Owner

Schön, danke euch.

Ich kenne folgenden Workflow:

  • Ihr gebt euer OK (am besten durch Daumen hoch, der auch in einer Übersicht zu sehen ist)
  • Ich habe den Pull Request (PR) erstellt und bin daher der "Kümmerer" für diesen. Das beinhaltet u. a.:
    • ich frage nach Reviews / Freigaben, wenn der PR länger nicht beachtet wird
    • ich reagiere zeitnah auf Kommentare, Verbesserungsvorschläge...
    • ich erstelle einen Merge-Commit - in diesem PR nach main (damit wird sich die Webseite demnächst "automagisch" aktualisieren)
    • ich schließe nach (eurer) Freigabe diesen PR

Forgejo unterstütz den Workflow nicht so wie ich ihn kenne.
Aber wir können unsere Arbeitsweise sicher an Forgejo anpassen.

Fragen? Dann bitte fragen! 😃

Schön, danke euch. Ich kenne folgenden Workflow: - Ihr gebt euer OK (am besten durch Daumen hoch, der auch in einer Übersicht zu sehen ist) - Ich habe den Pull Request (PR) erstellt und bin daher der "Kümmerer" für diesen. Das beinhaltet u. a.: - ich frage nach Reviews / Freigaben, wenn der PR länger nicht beachtet wird - ich reagiere zeitnah auf Kommentare, Verbesserungsvorschläge... - ich erstelle einen Merge-Commit - in diesem PR nach `main` (damit wird sich die Webseite _demnächst_ "automagisch" aktualisieren) - ich schließe nach (eurer) Freigabe diesen PR Forgejo unterstütz den Workflow nicht so wie ich ihn kenne. Aber wir können unsere Arbeitsweise sicher an Forgejo anpassen. Fragen? Dann bitte fragen! 😃
👍 1 🚀 1
sicking closed this pull request 2025-03-11 08:51:23 +00:00
sicking reopened this pull request 2025-03-11 08:51:31 +00:00
sicking merged commit 2f3c951b7d into main 2025-03-11 08:53:03 +00:00
sicking deleted branch bugfix/correct_blogartikel-oer-meets-fachdidaktik-aka-KLT 2025-03-11 08:53:03 +00:00
sicking commented 2025-03-11 08:57:53 +00:00
Author
Owner

Und wir brauchen einen besseren (automatisierten) Review-Prozess ;-)

user@pc  ~Dev/FOERBICO (main)
$hugo -s Website -d 'PROD'
Start building sites … 
hugo v0.139.0-dacef0df92a9a20072b4eaa42003e1a63ace8f41+extended linux/amd64 BuildDate=2024-11-18T16:17:45Z VendorInfo=gohugoio

Total in 18 ms
Error: error building site: process: readAndProcessContent: "Dev/FOERBICO/Website/content/posts/2025-02-25-KLT-Münster/index.md:65:1": failed to unmarshal YAML: yaml: line 65: found unexpected end of stream

Jetzt ist "main" kaputt. Das sollten wir möglichst vermeiden.
(alles nicht schlimm, nur ein sehr gutes Beispiel für das Ausprobieren unseren künftigen Workflows)

Und wir brauchen einen besseren (automatisierten) Review-Prozess ;-) ``` user@pc ~Dev/FOERBICO (main) $hugo -s Website -d 'PROD' Start building sites … hugo v0.139.0-dacef0df92a9a20072b4eaa42003e1a63ace8f41+extended linux/amd64 BuildDate=2024-11-18T16:17:45Z VendorInfo=gohugoio Total in 18 ms Error: error building site: process: readAndProcessContent: "Dev/FOERBICO/Website/content/posts/2025-02-25-KLT-Münster/index.md:65:1": failed to unmarshal YAML: yaml: line 65: found unexpected end of stream ``` Jetzt ist "main" kaputt. Das sollten wir möglichst vermeiden. (alles nicht schlimm, nur ein sehr gutes Beispiel für das Ausprobieren unseren künftigen Workflows)
sicking commented 2025-03-11 09:29:49 +00:00
Author
Owner

Weiter geht es im PR

Weiter geht es im PR https://git.rpi-virtuell.de/Comenius-Institut/FOERBICO/pulls/303
sicking pinned this 2025-03-12 07:31:53 +00:00
sicking commented 2025-03-12 07:33:40 +00:00
Author
Owner

Da dieser PR bereits gemerged ist, kann er (vermutlich deswegen) nicht "approved" werden.

Da dieser PR bereits gemerged ist, kann er (vermutlich deswegen) nicht "approved" werden.
sicking locked as Resolved and limited conversation to collaborators 2025-03-12 07:33:47 +00:00
Sign in to join this conversation.
No reviewers
phillipangelina
lmoessle
joerglohrer
No labels
Blog
CI
Community-Hub
Community-Hub/Anforderungen
Duplikat
Events
FAU
Fehler / Bug
GU
gutes erstes Issue
Hilfe gesucht
invalid
oer.community
Offene Frage
technik
Verbesserung
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Comenius-Institut/FOERBICO#300
No description provided.
Delete branch "bugfix/correct_blogartikel-oer-meets-fachdidaktik-aka-KLT"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?