Skip to content

Conversation

leakonvalinka
Copy link
Member

@leakonvalinka leakonvalinka commented Jul 28, 2025

This PR

  • adjusts the implementation to handle both paths that the disable-sync-metadata toggle introduces

Related Issues

relates: #1584
closes: #1509

Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
@leakonvalinka leakonvalinka changed the title feat(flagd): Adjust to disable-sync-metadata toggle feat(flagd): Adjust to disable-sync-metadata toggle in flagd Jul 29, 2025
Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
@leakonvalinka leakonvalinka marked this pull request as ready for review July 29, 2025 08:12
@leakonvalinka leakonvalinka requested a review from a team as a code owner July 29, 2025 08:12
Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
Copy link
Member

@aepfli aepfli left a comment

Choose a reason for hiding this comment

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

Thank you! Can't wait to fully get rid of the Metadata endpoint, maybe we should also mark the configuration syncMetadataDisabled as deprecated and to be removed in the next major version

Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
Signed-off-by: Konvalinka <lea.konvalinka@dynatrace.com>
@aepfli aepfli merged commit f1adc5d into open-feature:main Jul 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

7 participants