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

feat: onConfigurationChange #112

Merged

Conversation

typotter
Copy link
Collaborator


labels: mergeable

Eppo Internal
[//]: # (Link to the issue or doc corresponding to this chunk of work)
🎟️ Fixes ff-4116
📜 Design Doc

Motivation and Context

Pick up latest changes from common

Description

  • minor feat: onConfigurationChange

How has this been documented?

ff-4117

@typotter typotter changed the title Typo/ff 4116 java server sdk config change callback release feat: onConfigurationChange Mar 11, 2025
@typotter typotter requested review from aarsilv and greghuels March 11, 2025 19:21
Copy link
Collaborator

@aarsilv aarsilv left a comment

Choose a reason for hiding this comment

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

💪

@@ -271,6 +276,47 @@ public void testGetConfiguration() {
assertEquals(VariationType.NUMERIC, configuration.getFlagType("numeric_flag"));
}

@Test
public void testConfigurationChangeListener() throws ExecutionException, InterruptedException {
Copy link
Collaborator

Choose a reason for hiding this comment

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

💪

@typotter typotter enabled auto-merge (squash) March 13, 2025 14:42
@typotter typotter merged commit 848545a into main Mar 13, 2025
4 checks passed
@typotter typotter deleted the typo/ff-4116-java-server-sdk-config-change-callback-release branch March 13, 2025 14:43
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