Skip to content

Implement COM Kostenposition (and let Fremdkostenposition inherit from it)#286

Merged
hf-kklein merged 9 commits intomasterfrom
kostenposition
Jan 10, 2022
Merged

Implement COM Kostenposition (and let Fremdkostenposition inherit from it)#286
hf-kklein merged 9 commits intomasterfrom
kostenposition

Conversation

@hf-kklein
Copy link
Copy Markdown
Contributor

@hf-kklein hf-kklein commented Dec 22, 2021

Fixes #170
Fixes #285

⚠️ before merging into master please update the branch related #291 with the updates from this branch

@hf-kklein hf-kklein self-assigned this Dec 22, 2021
@hf-kklein hf-kklein requested a review from a team December 22, 2021 06:36
@hf-kklein hf-kklein marked this pull request as ready for review December 22, 2021 06:36
Schema for de-/serialization of Kostenposition
"""

class_name = Kostenposition
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Braucht man das für die Vererbung?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ne, das ist ein wiederentdecktes feature, das @hf-krechan ganz früher mal für BusinessObjects gebaut hat, aber dann in Vergessenheit geriet. Das erspart uns das Copy-Pasten der Post-load methode.
Ich habe das gestern überall eingebaut: #314

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ah uh, nice!

hf-aschloegl and others added 2 commits January 10, 2022 18:53
Co-authored-by: Annika <73470827+hf-aschloegl@users.noreply.github.com>
@hf-kklein hf-kklein merged commit f58b804 into master Jan 10, 2022
@hf-kklein hf-kklein deleted the kostenposition branch January 10, 2022 18:07
@hf-kklein hf-kklein restored the kostenposition branch January 10, 2022 18:07
@hf-kklein hf-kklein deleted the kostenposition branch January 10, 2022 18:09
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.

Why does COM Fremdkostenposition not inherit from COM Kostenposition? Implement COM Kostenposition

2 participants