Add Bankverbindung #884
Add Bankverbindung #884
Conversation
src/bo4e/com/bankverbindung.py
Outdated
| iban: Optional[str] = None | ||
|
|
||
| kontoinhaber: Optional[str] = None | ||
|
|
||
| bankkennung: Optional[str] = None | ||
|
|
||
| bankname: Optional[str] = None |
There was a problem hiding this comment.
Wäre ganz gut, hier docstrings zu haben. Insbesondere bei bankkennung. Soll das ein SWIFT Code sein?
src/bo4e/com/bankverbindung.py
Outdated
| if TYPE_CHECKING: | ||
| from ..enum.preismodell import Preismodell | ||
| from ..enum.rechnungslegung import Rechnungslegung | ||
| from ..enum.sparte import Sparte | ||
| from ..enum.vertragsform import Vertragsform | ||
| from .ausschreibungsdetail import Ausschreibungsdetail | ||
| from .menge import Menge | ||
| from .zeitraum import Zeitraum |
There was a problem hiding this comment.
welchen dieser importe verwenden wir hier überhaupt?
src/bo4e/com/bankverbindung.py
Outdated
| iban: Optional[str] = None | ||
| # International Bank Account Number = IBAN for example: DE07 1234 1234 1234 1234 12 | ||
|
|
||
| kontoinhaber: Optional[str] = None | ||
| # individual or entity in whose name a bank account is registered and who has legal rights over it | ||
|
|
||
| bankkennung: Optional[str] = None | ||
| # A unique code, such as a BIC (Bank Identifier Code) or SWIFT code, that identifies a specific bank in | ||
| # international transactions (e.g., BIC: DEUTDEFF for Deutsche Bank) | ||
|
|
||
| bankname: Optional[str] = None | ||
| # name of the bank e.g. Deutsche Bank |
There was a problem hiding this comment.
kannst du deutsche docstrings drauß machen?
also einmal mit : nach dem #und dann halt inhaltlich auf deutsch?
There was a problem hiding this comment.
hast du ein bsp für mich wie du es genau haben willst?
|
Damit das COM nicht im luftleeren Raum steht, sollte es entsprechend PR Hochfrequenz/BO4E-dotnet#466 auch noch zum Geschaeftspartner hinzugefügt werden. Hier: 5cff34e |
| @@ -193,6 +193,7 @@ | |||
| "Zaehlertyp", | |||
| "ZaehlertypSpezifikation", | |||
| "ZusatzAttribut", | |||
| "Bankverbindung", | |||
There was a problem hiding this comment.
Das sollte alphabetisch einsortiert sein
There was a problem hiding this comment.
Oder wird das hier irgendwie so autogeneriert?
There was a problem hiding this comment.
ne händisch, ich meine aber dass mir gesagt wurde dass das aber auto sortiert werden soll
|
Abgesehen von der Alphabetfrage passt es so für mich |
Im Moment wird das weder autogeneriert noch automatisch sortiert. Es ist auch nicht richtig sortiert, nur innerhalb von BOs, COMs und ENUMs. |
No description provided.