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

Schreibkorrekturen & Verbesserungen #167

Merged
merged 15 commits into from
May 9, 2020
Merged

Schreibkorrekturen & Verbesserungen #167

merged 15 commits into from
May 9, 2020

Conversation

crydotsnake
Copy link
Contributor

No description provided.

installer.md Outdated Show resolved Hide resolved
@alxndr-w
Copy link
Contributor

alxndr-w commented May 8, 2020

Wenn du hier noch weiter dran bist, dann lass ich den PR erstmal offen und du gibst Bescheid, wenn du alle Dateien durch hast :)

@crydotsnake
Copy link
Contributor Author

Mach ich! :)

@alxndr-w alxndr-w marked this pull request as draft May 8, 2020 14:26
@crydotsnake
Copy link
Contributor Author

Ich bin soweit Fertig @alexplusde

@crydotsnake crydotsnake marked this pull request as ready for review May 8, 2020 15:03
@crydotsnake
Copy link
Contributor Author

crydotsnake commented May 8, 2020

Jahr* 🤦

Ist vllt etwas unnötig.. aber dachte mir, es sieht schöner aus :).

@crydotsnake crydotsnake changed the title Schreibkorrekturen Schreibkorrekturen & Verbesserungen May 8, 2020
Copy link
Contributor

@alxndr-w alxndr-w left a comment

Choose a reason for hiding this comment

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

Hatte es mit Language Tool überflogen. Da geht noch was. Ich lass den PR offen und werde noch Ergänzungen beisteuern.

@crydotsnake
Copy link
Contributor Author

Okay. Kann ich noch irgendwas tun?. Habe leider kein Setup für das Language Tool gefunden.

@alxndr-w
Copy link
Contributor

alxndr-w commented May 8, 2020

Hast du VSCode oder welchen Editor nutzt du?

@crydotsnake
Copy link
Contributor Author

VSCode

@alxndr-w
Copy link
Contributor

alxndr-w commented May 9, 2020

@skerbis ich würde diesen Stand gerne mal mit reinnehmen. Es ist noch super viel zu tun von den Dateien mit den Anfangsbuchstaben A-N, denn ich habe irgendwann zeitlich aufgeben müssen.

Achtung, mein Markdown-Linter hat Autokorrekturen vorgenommen, falls es hier Probleme gibt, müssen wir leider reverten oder schnell das Problem fixen.

@crydotsnake
Copy link
Contributor Author

@alexplusde Ohje.. Ich habe vorhin vergessen, deine änderungen erst zu pullen.. Hoffe ich hab jetzt nichts an meinem PR oder Fork kaputt gemacht....

@crydotsnake
Copy link
Contributor Author

Scheint nichts kaputt zu sein... habe wohl nur deine änderungen nochmal gepusht. Muss daran denken zu pullen, bevor ich meine änderungen commite und pushe 🤦.

Copy link
Contributor

@skerbis skerbis left a comment

Choose a reason for hiding this comment

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

Aufzählungen sollten grundsätzlich klein beginnen. Es sei denn es handelt sich beim ersten Wort um ein Substantiv oder um einen Satz.

Anders verhält es sich bei Aufzählungen. Hier werden nur vollständige Sätze mit Großbuchstaben begonnen, oder wenn es sich beim ersten Wort um ein Substantiv handelt

module.md Outdated
@@ -26,7 +26,7 @@ Jedes Modul hat einen Eingabeteil, der mit verschiedensten Eingabeelementen vers

Eingabeelemente können zum Beispiel sein:

- ein oder mehrere Textfelder
- Ein oder mehrere Textfelder
Copy link
Contributor

Choose a reason for hiding this comment

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

das "ein" sollte kleingeschrieben werden.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay. hab es wieder abgeändert.

@alxndr-w
Copy link
Contributor

alxndr-w commented May 9, 2020

Bin mir nicht sicher, aber ich glaube, da ist was kaputt gegangen mit head/commit

@crydotsnake
Copy link
Contributor Author

Wie kann man sehen ob was kaputt gegangen ist..

@crydotsnake crydotsnake requested a review from skerbis May 9, 2020 15:32
@crydotsnake
Copy link
Contributor Author

Ja... Jetzt sehe ich es auch.. habe mir die Dateien angeschaut...

Wenn du magst close ich den PR, Forke das Docs repo komplett neu, und ändere nochmal alles... @alexplusde. Ärgert mich gerade richtig, das ich nicht aufgepasst habe...

@alxndr-w
Copy link
Contributor

alxndr-w commented May 9, 2020

Ne, deinen letzten commit reverten, dann frisch pullen und dann nochmal machen. PR schließen hilft da nicht.

Alles kein großes Problem, Übung macht den Meister.

@crydotsnake
Copy link
Contributor Author

Sry, was soll ich machen?. Habs nicht ganz verstanden 😕

@crydotsnake
Copy link
Contributor Author

Hm.. Hat auch nicht geklappt 😕

@crydotsnake
Copy link
Contributor Author

@alexplusde Ich glaub, ich habs geschafft. Könntest du eventuell überprüfen, ob ich alles richtig gemacht habe?.

@crydotsnake
Copy link
Contributor Author

Ich gehe die ganzen Dateien nochmal durch.

@crydotsnake
Copy link
Contributor Author

@alexplusde Ich bin nochmal alles durchgegangen, und habe die jeweiligen fehler behoben. Sollte noch etwas nicht schön aussehen, oder wenn ich noch etwas ändern soll kümmere ich mich drum! 😊

@skerbis skerbis requested a review from alxndr-w May 9, 2020 18:39
@skerbis skerbis removed their request for review May 9, 2020 18:41
@alxndr-w alxndr-w merged commit ab65861 into redaxo:master May 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants