Skip to content
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

fix: Serialize Geraetemerkmal WITH GAS_ prefix in STJ (instead of w/o) (on v0.8.x legacy branch) #619

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

hf-kklein
Copy link
Collaborator

@hf-kklein hf-kklein commented Dec 4, 2024

this cherrypicks the changes from #618 to the v0.8.x legacy branch (used in 2 of our services ;))

@hf-kklein
Copy link
Collaborator Author

@hf-fvesely nach dem mergen bitte ein release 0.8.8 auf dem entsprechenden branch erstellen
{5BF4D7F8-AEB0-46C6-8BE9-1DE4FDE0603F}

so kriegst du die fixes in die services ohne direkt die minor version auf 0.20 zu bumpen.

@@ -120,6 +142,21 @@ public void TestSystemText_Success_NonNullable()
result.Merkmal.Should().Be(Geraetemerkmal.GAS_G4);
Copy link
Contributor

Choose a reason for hiding this comment

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

Hier ist noch eine Verwirrung zwischen gas und wasser

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ne, da kommt nur die github ansicht nicht drauf klar.
{9B13D25E-0E7B-4AF4-A25F-C5F3F1DA8C43}

der test hat nichts mit wasser zu tun.

@hf-kklein hf-kklein requested a review from JoschaMetze December 5, 2024 11:47
@hf-fvesely hf-fvesely merged commit 1ed5ec1 into v0.8.x-legacy Dec 10, 2024
9 checks passed
@hf-fvesely hf-fvesely deleted the gas_g4_08x branch December 10, 2024 09:24
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.

2 participants