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

Documentation: Refactoring of plugins.md #690

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

rhuss
Copy link
Contributor

@rhuss rhuss commented Dec 15, 2024

Hier mein erster Wurf für eine Restrukturierung von plugins.md. Dabei liegt der Fokus auf der Kapitelstruktur, es sind noch einige Gaps offen:

  • Beispiele
  • Tabellen mit allen Werten
  • Englische Version

Out-of-scope in diesem PR:

  • Beschreibung der einzelnen Plugins
  • Detailierte Beschreibung von Pipelines

Beim Review koennt ihr gerne direkt korrigieren und auf den Branch pushen, er ist offen für Maintainers (vielleicht bei groesseren Aenderung noch einen Review Kommentar).

Random Opinion following ..., nicht wichtig :-) Darueberhinaus habe ich noch ein paar kleine Anmerkungen (sorry, ich kann nicht anders :-), die aber für den PR nicht relevant sind:
  • "Plugin" ist eine etwas unglueckliche Bezeichnung, das sie in vielen anderen Faellen, externen code und Konventionen bezeichnen, die ausserhalb des Kernprojekts leben und vorgegebene APIs implementieren. Im Falle von evcc ist aber die Liste der Plugins fix und koennen nicht selbst ergaenzt werden. Z.b. muesste man für eine Resol VBus Integrationen den core-code erweitern. In meinen Augen handelt es sich vielmehr um Integrationen, bei der externe Sources (oder Producer) und Sinks (oder Consumer) eingebunden werden, so wie sie z.b. in dem klassischen EAI Patterns Buch beschrieben sind. Mir ist klar das im Kontext von evcc "Integrationen" anders interpretiert wird (i.e. wie man evcc in andere Systeme integrieren kann, also mehr eine "API" Beschreibung), daher sollten man es auch so lassen. Ich will hier keine Diskussion starten, sondern mein innerer Spiesser wollte das einfach loswerden :-P und evtl. hilft das in Diskussionen um Missverstaendnisse zu vermeiden.

  • Das "combined status" plugin passt in meinen Augen nicht wirklich in die Liste, da es nicht für alle Geraete genutzt werden kann (für meter macht das keinen Sinn, oder ?). Vielmehr wuerde ich es als Eigenschaft von vehicle sehen, bei denen Plugins genutzt werden koenne. Your mileage may vary.

@rhuss
Copy link
Contributor Author

rhuss commented Dec 15, 2024

In dem PR verwende ich ds one sentence per line pattern, was es bei PR request reviews (und bei aenderungs vorschlaegen) deutlich einfacher macht. Für das Rendern macht es keinen Unterschied.

@naltatis
Copy link
Member

In dem PR verwende ich ds one sentence per line pattern, was es bei PR request reviews (und bei aenderungs vorschlaegen) deutlich einfacher macht. Für das Rendern macht es keinen Unterschied.

Das ist gut. Das machen wir auch so.

@naltatis naltatis self-assigned this Dec 15, 2024
@naltatis naltatis added the enhancement New feature or request label Dec 15, 2024
@rhuss
Copy link
Contributor Author

rhuss commented Dec 16, 2024

@naltatis gibt es eine Moeglichkeit, labels in der Dokumentation darzustellen ? Ich wuerde gerne zwei labels fur lesen und schreiben jeweils bei den Plugins anzeigen (anstelle von textuell (lesen / schreiben), was nicht sehr auffaellt)

@naltatis
Copy link
Member

Ich bin den aktuellen Stand gestern einmal durchgegangen und hab dabei ein paar sprachliche Dinge mit angepasst. Die Struktur und der Erklärungslevel gefällt mir total gut. Willst du das Umhängen /docs/[reference > devices]/plugins in diesem PR gleich mit machen? Wäre für mich dann der unterste Punkt im "Geräte" Abschnitt.

Zum "Plugin" Namen: Ich verstehe die Argumentation. Das ist nur aktuell schon an so vielen Stellen in Code und Sprachgebrauch, dass das schwer zu ändern ist ohne unnötige Verwirrung zu erzeugen.

@naltatis
Copy link
Member

@rhuss Für die Darstellung könntest du kbd oder mark Tags verwenden. Styling dafür bringt Docosaurus bereits mit. Standard Markdownsyntax gibts dafür nicht. Via Plugins ginge da was. Aber HTML Tags schreiben ist aktuell auch erlaubt.

... mit Endgeräten spricht (<kbd>lesen</kbd>/<kbd>schreiben</kbd>).
... von Daten genutzt werden <mark>(lesen)</mark>.
Bildschirmfoto 2024-12-16 um 08 53 08

Optisch finde ich den kbd Style passender, weil der abgeschlossener wirkt. Semantisch passen beide nicht wirklich. Aber lass uns doch mit kbd starten.

@rhuss
Copy link
Contributor Author

rhuss commented Dec 16, 2024

@naltatis ich hab mal meine verstaubten CSS Kenntnisse ausgegraben und eine simple React Komponente Tag.js eingebaut, um die Labels zu rendern. Die Farbgebung & Spacing kann man gerne noch anpassen, verlasse mich da aber auf die Experten ;-)

image

@naltatis
Copy link
Member

Hab das etwas angepasst und das vorhanden Farbschema verwendet.

Bildschirmfoto 2024-12-16 um 21 05 39

@rhuss
Copy link
Contributor Author

rhuss commented Dec 16, 2024

Ich habe ein die Beispiele zu meter, charger und vehicle eingefuegt. Dabei habe ich mich an den Templates orientiert, kann aber nicht sagen ob meine Beispiele korrekt umgesetzt sind (oder etwas mandatory parameter fehlen), bzw. ob sie ueberhaupt Sinn machen. Es waere toll wenn ihr darueber schauen koenntet und ggfs. durch sinnvollere Beispiele ersetzt.

Als naechstes werde ich versuchen, die Tabellen sinnvoll zu ergaenzen. Da braeuchte ich spaeter dann ebenfalls die Hilfe von euch SMEs :)

@rhuss
Copy link
Contributor Author

rhuss commented Dec 16, 2024

Willst du das Umhängen /docs/[reference > devices]/plugins in diesem PR gleich mit machen? Wäre für mich dann der unterste Punkt im "Geräte" Abschnitt.

Das wuerde ich dann am Ende machen wenn wir den Inhalt soweit festgezurrt haben.


Das Schema hat dabei immer folgende Struktur:

```yaml {3,5-6,8}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@naltatis Hast Du eine Idee, wie ich die Platzhalter (<...>) hervorheben kann ? Ich habs mit Prism plugins probiert, aber React/Docsaurus-Fu ist nicht gut genug. Idealerweise waeren die Platzhale in kursiv + bold.

Ist aber nicht dringend, man kann sicher auch ohne leben. Nur falls Du zufaellig was weisst.

Testweise habe ich line-highlight Prism plugin enabled, aber so toll ist das hier auch nicht (sollte man wohl auch mit magic comments)

Copy link
Member

Choose a reason for hiding this comment

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

Line-Highlights mit Magic Comments sollten ja ohne Konfigurationsänderung schön möglich sein:

tariffs:
  currency: EUR # (default EUR)
  grid:
    // highlight-start
    type: fixed
    // highlight-end
    price: 0.294 # EUR/kWh

Aber du möchtest nur die Teilwörter, keine Zeilen richtig? Da bin ich auch überfragt.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, genau. Ich wollte die Platzhalter attr1 bzw attr2 hervorheben.
Aber kein Problem, das ist nur Nebensache. Inhalt ist wichtiger ;-)

Danke fuers Feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants