Skip to content

Conversation

@FreddyFox892
Copy link
Collaborator

No description provided.

@hf-kklein hf-kklein changed the title Add New Enums Add BO Einspeisung Sep 9, 2024
Copy link
Contributor

@hf-kklein hf-kklein Sep 9, 2024

Choose a reason for hiding this comment

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

was hat die profilart mit dem BO Einspeisung zu tun?
ich weiß, ich hab selbst den PR umbenannt, aber die klammer um die änderungen ist doch das BO Einspeisung? und da fällt die profilart aus der reihe

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Stimmt, aber war in dem .net pr so mit drin

Copy link
Contributor

Choose a reason for hiding this comment

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

dann löschen wir es hier raus und machen einen eigenen PR? sonst ist es ja komisch zu reviewen

Copy link
Contributor

Choose a reason for hiding this comment

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

Bin auch für erstmal löschen, da es so ja derzeit nicht in der lib genutzt wird. Und den docstring müssten wir dann auh eh nochmal neu schreiben, aber dann erst wenn wir das COM Lastprofil auch einbauen

@hf-kklein
Copy link
Contributor

Codemäßig hab ich zwei drei Sachen gefunden.
Für die inhaltiche Review ist das Gremium da @bo4e/the-gremium

"""

AUSFALLVERGUETUNG = "AUSFALLVERGUETUNG" #: Ausfallvergütung
# Ausfallvergütung für den Fall, dass andere Vermarktungsmethoden nicht verfügbar sind
Copy link
Contributor

Choose a reason for hiding this comment

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

Die Kommentare sind hier doppelt, also ich meine jeweils das obere: #: Ausfallvergütung braucht man nicht auch noch, wenn man die Langfassung hat, oder ich habe irgendeine Dokuemtationsänderung nicht mitbekommen. :-)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@lord-haffi lord-haffi Sep 9, 2024

Choose a reason for hiding this comment

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

Bitte sicherstellen, ob der Kommentar in der Doku auftaucht. Bin mir relativ sicher, dass das Autodoc Feature von Sphinx nur #: über und in der Zeile und """ ... """ unter der Zeile kann.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah sehr gut zu wissen, ja stimmt so hatten wir es an anderen Stellen glaub ich auch gehandhabt, ich hab bisher einfach immer geschaut wie es an anderen Stellen gemacht wird

@hf-kklein hf-kklein requested a review from a team September 9, 2024 12:39
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.

Add Einspeisung BO Object and Enums (EEGVermarktungsform, FernsteuerbarkeitStatus)

5 participants