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

Doku-Links in JVerein zeigen jetzt auf die OpenJVerein-Doku #55

Merged
merged 4 commits into from Nov 24, 2023

Conversation

NicoB77
Copy link

@NicoB77 NicoB77 commented Oct 28, 2023

Es wäre gut, wenn jemand versuchen könnte, sich wie in CONTRIBUTING.md beschrieben bei GitBook anzumelden, damit wir testen, ob es wirklich funktioniert. Sonst muss ich Einladungen verschicken, damit ich nicht der einzige mit Zugang zu GitBook bin.

Ich habe die Dokumentation an die Änderungen im letzten Release angepasst.

tschubert and others added 2 commits October 28, 2023 19:43
The links did not yet reflect the new structure of the online documentation.
I updated the links to the best of my knowledge/understanding.
@MSchmalzl
Copy link

MSchmalzl commented Oct 29, 2023

Ich wäre bei GitBook jetzt angemeldet. Bei der Anmeldung habe ich mich für die Anmeldung mit meinem GitHub Account entschieden in welchem ich auch Mitglied der OpenJVerein Organisation bin.

In meinem GitHub Account unter Settings --> Integrations --> Applications --> Authorized GitHub Apps taucht GitBook auch auf.

Zugriff auf das OpenJVerein GitBook habe ich allerdings nicht. Zumindest finde ich weder in GitHub noch in GitBook etwas. Somit ist es vermutlich notwendig dass für dieses Repo Einladungen verschickt werden.

Ich glaub das liegt daran dass das GitBook ein eigenes Repo ist. Gibt es nicht auch die möglichkeit das GitBook ins OpenJVerein Repo zu integrieren?

@MSchmalzl
Copy link

Ich habe jetzt mal das GitBook geforket und versucht in GitBook eine synchronisierung einzurichten. Wäre das so der richtige weg?

@NicoB77
Copy link
Author

NicoB77 commented Oct 29, 2023

Ich habe jetzt in der Member-Ansicht von https://github.com/openjverein ein Readme mit einem Einladungslink für die GitBook-Organisation angelegt. CONTRIBUTING.md passe ich noch an.

@NicoB77
Copy link
Author

NicoB77 commented Oct 29, 2023

Ich habe jetzt mal das GitBook geforket und versucht in GitBook eine synchronisierung einzurichten. Wäre das so der richtige weg?

Im Prinzip ja, aber Du solltest mit Deinem Fork synchronisieren. Du hast zwei Dateien des Handbuchs verändert, das habe ich revidiert.

@MSchmalzl
Copy link

MSchmalzl commented Oct 29, 2023

Dass da was nicht korrekt ist habe ich schon fast vermutet. Mir wurde nämlich nur das OpenJVerein GitBook angezeigt. Mein eigenes leider nicht.

Habe jetzt nochmal auf App installieren geklickt. Jetzt wird zumindest mein Fork angezeigt.

Auch kann ich mir nicht erklären wie die Änderung zustande kam. Denn außer einem Fehler beim synchronisieren wurde mir kein GitBook angezeigt.

Derzeit scheint der Server recht ausgelastet zu sein. Ich versuche ein anderes mal mein Glück.

@MSchmalzl
Copy link

MSchmalzl commented Nov 6, 2023

In Datei JahresabschlussListView.java müsste Zeile 40 durch folgendes ersetzt werden:
DokumentationUtil.JAHRESABSCHLUSS, false, "question-circle.png");

@MSchmalzl
Copy link

MSchmalzl commented Nov 12, 2023

Ich hätte hier noch ein paar grundsätzliche Fragen zu Gitbook.

Ich bin ja jetzt dieser Organisation beigetreten. Somit taucht ja dann bei mir das JVerein Handbuch auf. Soll dann bei Änderungen ein Change-Request gestartet werden oder ein PR über das eigene Repo?

Weiß jemand wie man einen Change Request wieder löscht?

Ist die Erreichbarkeit der Server bei euch auch so beschissen? Unter der Woche geht es besser. Von Freitag Nachmittag bis Sonntag Abend so gut wie gar nicht.

@willuhn willuhn merged commit eae0c3b into openjverein:master Nov 24, 2023
@NicoB77 NicoB77 deleted the pr_doku branch December 1, 2023 16:10
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

5 participants