Skip to content

Fix 1661#1662

Merged
ryanjbaxter merged 172 commits intospring-cloud:mainfrom
wind57:fix-1661
Jun 10, 2024
Merged

Fix 1661#1662
ryanjbaxter merged 172 commits intospring-cloud:mainfrom
wind57:fix-1661

Conversation

@wind57
Copy link
Copy Markdown
Contributor

@wind57 wind57 commented Jun 1, 2024

No description provided.

wind57 and others added 30 commits December 4, 2021 07:59
wind57 added 17 commits April 12, 2024 18:37
@wind57 wind57 marked this pull request as ready for review June 1, 2024 10:53
@wind57
Copy link
Copy Markdown
Contributor Author

wind57 commented Jun 3, 2024

merged latest main, after the release last week. @ryanjbaxter this one is ready

@wind57
Copy link
Copy Markdown
Contributor Author

wind57 commented Jun 6, 2024

@ryanjbaxter friendly reminder...

@amb-damian-kostrzewski-pt
Copy link
Copy Markdown

What are you waiting for? We are waiting for this fix to get rid of the workaround.

@@ -106,7 +106,7 @@ public void setNamespace(String namespace) {
}

public String getNamespace(String defaultValue) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We also noticed it...
Maybe this method should be renamed to getNamespaceOrDefault ?

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.

it can't be renamed since it is public

@amb-szymon-kuna-pt
Copy link
Copy Markdown

@ryanjbaxter when you gonna release this fix?

@ryanjbaxter
Copy link
Copy Markdown
Contributor

Sorry for the delay. Other work is taking priority at the moment

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.

Fabric8 LeaderProperties.getNamespace(defaultValue) no longer falls back to given default when namespace is undefined

6 participants