-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add BO Einspeisung #883
Open
FreddyFox892
wants to merge
9
commits into
main
Choose a base branch
from
FST/Issue878_UpdateBO_Enums
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+118
−0
Open
Add BO Einspeisung #883
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
c305592
First try
FreddyFox892 9e582ae
.-
FreddyFox892 1a90517
add test
FreddyFox892 c69f49c
..
FreddyFox892 81361e5
ka
FreddyFox892 e053ffc
fa
FreddyFox892 deb0bb5
Update src/bo4e/enum/fernsteuerbarkeit_status.py
FreddyFox892 d40e2bc
rmv profilart
FreddyFox892 b74faaa
Merge branch 'main' into FST/Issue878_UpdateBO_Enums
FreddyFox892 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
rmv profilart
commit d40e2bca703ca10fe7129b3ad6457c911f3f1e56
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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