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.

Sign up for GitHub

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

Jump to bottom

Changes on messaging layer configuration on UI not persisted. #2112

Closed
obermeier opened this issue Oct 30, 2023 · 1 comment · Fixed by #2114
Closed

Changes on messaging layer configuration on UI not persisted. #2112

obermeier opened this issue Oct 30, 2023 · 1 comment · Fixed by #2114
Assignees
@dominikriemer
Labels
bug Something isn't working
Milestone
0.93.0

Comments

@obermeier
Copy link
Member

Apache StreamPipes version

dev (current development state)

Affected StreamPipes components

Backend, UI

What happened?

Changes on the page /configuration/messaging E.g. host of Nats are lost if I leve the page.

Changing the value in the db works.

How to reproduce?

  • Change value nats to nats.example.com
  • Click the UPDATE button.
  • Switch to another page.
  • Return to /configuration/messaging
    The value is now nats

Expected behavior

No response

Additional technical information

No response

Are you willing to submit a PR?

None
@obermeier obermeier added the bug Something isn't working label Oct 30, 2023
@bossenti bossenti added this to the 0.95.0 milestone Oct 31, 2023
@dominikriemer dominikriemer modified the milestones: 0.95.0, 0.93.0 Oct 31, 2023
dominikriemer added a commit that referenced this issue Oct 31, 2023
@dominikriemer
fix(#2112): Properly update broker configs
307c16f
@dominikriemer dominikriemer self-assigned this Oct 31, 2023
@dominikriemer dominikriemer mentioned this issue Oct 31, 2023
Merged
@dominikriemer dominikriemer linked a pull request Oct 31, 2023 that will close this issue
fix(#2112): Properly update broker configs #2114
Merged
@dominikriemer
Copy link
Member

Thanks @obermeier for finding this.

dominikriemer added a commit that referenced this issue Nov 1, 2023
@dominikriemer
fix(#2112): Properly update broker configs (#2114)
79e7d18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dominikriemer dominikriemer

Labels
bug Something isn't working
Projects
None yet
Milestone
0.93.0
Development

Successfully merging a pull request may close this issue.

fix(#2112): Properly update broker configs apache/streampipes
3 participants
@obermeier @dominikriemer @bossenti