Skip to content

Commit bcafbc0

Browse files
send config update notification before unblocking (#349)
1 parent 9062dd8 commit bcafbc0

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

OptimizelySDK/Config/PollingProjectConfigManager.cs

+4-5
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/*
1+
/*
22
* Copyright 2019-2020, 2022-2023 Optimizely
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
@@ -25,7 +25,7 @@
2525
namespace OptimizelySDK.Config
2626
{
2727
/// <summary>
28-
/// Abstract class that implements ProjectConfigManager interface and provides
28+
/// Abstract class that implements ProjectConfigManager interface and provides
2929
/// basic scheduling and caching.
3030
/// Instances of this class, must implement the <see cref="Poll()"/> method
3131
/// which is responsible for fetching a given ProjectConfig.
@@ -181,11 +181,10 @@ public bool SetConfig(ProjectConfig projectConfig)
181181
CurrentProjectConfig = projectConfig;
182182
SetOptimizelyConfig(CurrentProjectConfig);
183183

184-
// SetResult raise exception if called again, that's why Try is used.
185-
CompletableConfigManager.TrySetResult(true);
186-
187184
NotifyOnProjectConfigUpdate?.Invoke();
188185

186+
// SetResult raise exception if called again, that's why Try is used.
187+
CompletableConfigManager.TrySetResult(true);
189188

190189
return true;
191190
}

0 commit comments

Comments
 (0)