Add Erreichbarkeit #885
Add Erreichbarkeit #885
Conversation
| startzeit: Optional[time] = None | ||
| endzeit: Optional[time] = None |
There was a problem hiding this comment.
kannst du einen test hinzufügen, wie das serialisiert wird?
das wäre wichtig für die kompatabilität
There was a problem hiding this comment.
der scheint probleme haben das auf github mit der Time funktion zu encode
Lokal läuft das durch
There was a problem hiding this comment.
also was mich interessiert ist: sieht 17uhr hier in python gleich aus wie 17uhr drüben in c#?
There was a problem hiding this comment.
der scheint probleme haben das auf github mit der Time funktion zu encode
tests/test_erreichbarkeit.py:16: in TestErreichbarkeit
montag_erreichbarkeit=Zeitfenster(
E pydantic.errors.PydanticUserError:Zeitfensteris not fully defined; you should defineZusatzAttribut, then callZeitfenster.model_rebuild().
E
E For further information visit https://errors.pydantic.dev/2.8/u/class-not-fully-defined
@lord-haffi weiß vllt mehr.
There was a problem hiding this comment.
Die neuen Modelle müssen der init.py hinzugefügt werden
There was a problem hiding this comment.
@lord-haffi es ligt nicht daraan das es nicht in der init war
| @@ -82,3 +83,4 @@ class Geschaeftspartner(Geschaeftsobjekt): | |||
| #: Todo: Add optional connection to marktteilnehmer as discussed in workshop | |||
| #: not clear what is the best solution here - circular import marktteilnehmer? | |||
| #: discussed in workshop on Feb 6 2024: yes we need the bidirectional option, let's figure out a solution somehow. | |||
| erreichbarkeit: Optional[Erreichbarkeit] = None | |||
There was a problem hiding this comment.
Die Erreichbarkeit und auch auch die Bankverbindung und Steuernummer gerne noch im Geschäftspartner-Test ergänzen. Fällt nur nicht auf, weil alles optional ist.
No description provided.