beackport 8.5: Feature/reorganize properties (#662) #690
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* deprecate old properties * refactoring usage of properties * wip * wip * wip * wip * legacy config still working * removed unused configuration * added backend issuer url to default saas props * adjusted readme * fixed broken test * functionality of new property strategy proved * moved from spring profiles to properties post processor * moved tests as well * worker overrides are applied over global overrides * typo * table format * add a general comment for first orientation * removed property * typo * updated docs about variable fetching * typo * update link to reflect new properties class * typo * deprecated classes * updated executorserviceconfig to reflect new properties * added a test to assert all deprecations are applied correctly * applied deprecation notices to all legacy properties * adjusted a reason comment (cherry picked from commit ef37537)
jonathanlukas
changed the title
|
@jonathanlukas Should we perhaps include it in 8.6 as the first supported version? I'd rather not change the configuration on the connectors side at the moment and adopt this in the 8.6-alpha1. |
|
I can relate to this. But, the new properties structure should still work for old properties. Is there any way to find this new properties structure in the camunda sdk? |
chillleader
approved these changes
Mar 21, 2024
This was referenced Apr 5, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
deprecate old properties
refactoring usage of properties
wip
wip
wip
wip
legacy config still working
removed unused configuration
added backend issuer url to default saas props
adjusted readme
fixed broken test
functionality of new property strategy proved
moved from spring profiles to properties post processor
moved tests as well
worker overrides are applied over global overrides
typo
table format
add a general comment for first orientation
removed property
typo
updated docs about variable fetching
typo
update link to reflect new properties class
typo
deprecated classes
updated executorserviceconfig to reflect new properties
added a test to assert all deprecations are applied correctly
applied deprecation notices to all legacy properties
adjusted a reason comment
(cherry picked from commit ef37537)