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

Handbuch Erste Schritte Teil 1 #14

Closed
wants to merge 33 commits into from

Conversation

tenorix
Copy link
Contributor

@tenorix tenorix commented Jan 15, 2023

Hallo,
ich habe begonnen, die Einführung/Erste Schritte zu schreiben. Dazu schon hier mal der erste Teil als Pull Request. Hier das Preview: https://tenorix.github.io/portfolio-help/ .

Als Konvention habe ich mich für den "du"-Stil entschieden.

Ich habe dazu in der index.md auf der obersten Ebene ziemlich umgebaut, ich hoffe, das ist so okay.

Ich musste ein paar Änderungen an der mkdocs.yml und CNAME machen, damit das bei mir im Repository über die Pipe produziert werden kann. Keine Ahnung, wie ich vermeide, dass das hier mit übergeben wird.

Viele Grüße
Martin

@tenorix
Copy link
Contributor Author

tenorix commented Jun 2, 2023

Hallo,
mir fehlt hier ein Signal. Ich kann nicht einordnen, was der ausstehende Pull bedeutet. a) passt nicht b) super, aber wo bleibt Teil 2. Ich würde ja auch gerne was draus lernen.
Viele Grüße Martin

@Morpheus1w3
Copy link

@buchen @Nirus2000

@Nirus2000
Copy link
Member

Nirus2000 commented Jun 30, 2023

Hallo @tenorix
Sehr gut bisher... ich habe lediglich nur die Rechtschreibung korrigiert... alle können durch dich und dem „akzeptieren“ (sind in Pending) abgeändert werden.
Die Video's gefallen mir... bitte weiter so und versuchen ein einheitliches Bildformat (nur Ausschnitt von PP) ohne Desktop.
(siehe https://tenorix.github.io/portfolio-help/intro-wertpapiere-anlegen/)
Super.... i like it !!

Als Hilfe... kannst du unter https://rechtschreibpruefung24.de/ deine Texte korrigieren lassen... ;-)

@tenorix
Copy link
Contributor Author

tenorix commented Jul 1, 2023

Hallo,
danke für eurer Feedback. Hier die Liste, die ich noch abarbeiten werde.

Inhaltlich:

Aus Pull Request entfernen - meine Anpassungen waren für die mkdocs Produktion in meinem Repository und gehören nicht nach buchen/portfolio-help:

  • .github/workflows/main.yml
  • .gitignore
  • README.md
  • docs/CNAME

Änderungen entfernen, die nicht nach buchen/portfolio-help übergeben werden sollen:

  • mkdocs.yml

@Nirus2000 : Du schreibst "... alle können durch dich und dem „akzeptieren“ (sind in Pending) abgeändert werden." Hmm, das klingt so, als ob du Änderungsvorschläge gemacht hättest. Aber ich finde die nicht 😕 - wie gesagt, meine ersten Schritte in github.

  • Korrekturen von Nirus2000 angenommen (siehe unten)

Ist lokal für mein Dev-Repo und soll nicht im Pull-Request übergeben werden
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
docs/intro-bestaende-erstmalig-aufbauen.md Outdated Show resolved Hide resolved
docs/intro-bestaende-erstmalig-aufbauen.md Outdated Show resolved Hide resolved
docs/intro-neue-portfoliodatei-anlegen.md Outdated Show resolved Hide resolved
docs/intro-wertpapiere-anlegen.md Outdated Show resolved Hide resolved
Martin and others added 8 commits July 1, 2023 14:59
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
tenorix and others added 8 commits July 1, 2023 15:48
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
Co-authored-by: Nirus2000 <45203494+Nirus2000@users.noreply.github.com>
buchen pushed a commit that referenced this pull request Jul 2, 2023
Issue: #14
Signed-off-by: Martin G <85380943+tenorix@users.noreply.github.com>
[squashed commits; rebased to master]
Signed-off-by: Andreas Buchen <andreas.buchen@gmail.com>

patch
@buchen
Copy link
Member

buchen commented Jul 2, 2023

Hallo @tenorix,

vielen, vielen Dank für die Contribution. Die Hilfeseiten (und die Dokumentation im allgemeinen) sind wirklich ein Stiefkind. Ich bin einfach kein Schreiberling...

Mir gefällt die Struktur super gut. Mit den "ersten Schritten" weiß man viel eher wo man einsteigen soll. Auch finde ich die Benennung der Kapitel sehr gut. Die bisherigen Kapitel sind ja eher Detailinformationen mit denen man am Anfang gar nichts anfangen kann.

Ich habe mir erlaubt Deine Änderungen alle zusammen in einen einzelnen Commit zusammenzuführen (squash) und in den main branch zu mergen. Damit sind sie erst mal live. Wenn Dir noch was auffällst kannst Du einfach einen neuen Pull Request auf machen. Am einfachsten ist das immer, wenn Du einen neuen Feature branch aufmachst, der auf den "master" branch hier zeigt. Lokal machst Du dann gff. ein rebase, aber eher keinen merge - das macht es einfacher mit den pull requests.

Ich habe noch ein paar Kleinigkeiten geändert - ein Rechtschreibfehler hier, ein doppeltes "und" da. Die habe ich nicht als eigenständiges Commit aufgenommen sondern in Deinen Change aufgenommen. Hier ist noch der Patch - falls Dich interessiert was genau.
Patch.patch

Ping mich ggf. unter portfolio dot performance dot help at gmail dot com an wenn ich einen pull request hier nicht sofort sehe.

buchen pushed a commit that referenced this pull request Jul 2, 2023
Issue: #14
Signed-off-by: Martin G <85380943+tenorix@users.noreply.github.com>
[squashed commits; rebased to master]
Signed-off-by: Andreas Buchen <andreas.buchen@gmail.com>
@buchen
Copy link
Member

buchen commented Jul 2, 2023

Anmerkung: Es gibt auch ein kleines + Zeichen links über "alle Wertpapiere", das ist aber nicht das richtige.

Das ist ein guter Hinweise. Ich habe auch schon überlegt, ob ich nicht in das Kontextmenü dort das Anlegen von neuen Wertpapiere ebenfalls hinzufüge. Wie Du vielleicht gesehen hast, habe ich in der letzten Version das "Neu..." Menü schon erweitert.

@buchen buchen closed this Jul 4, 2023
Sn1kk3r5 added a commit that referenced this pull request Apr 29, 2024
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.

None yet

4 participants