Skip to content

Commit 44272fa

Browse files
msohailhussainMichael Ng
authored andcommitted
polling config manager should be started by default. (#199)
1 parent 7a67e94 commit 44272fa

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

OptimizelySDK.Tests/OptimizelyTest.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2908,6 +2908,7 @@ public void TestDFMNotificationWhenProjectConfigIsUpdated()
29082908
.WithSdkKey("QBw9gFM8oTn7ogY9ANCC1z")
29092909
.WithLogger(LoggerMock.Object)
29102910
.WithPollingInterval(TimeSpan.FromMilliseconds(1000))
2911+
.WithStartByDefault(false)
29112912
.WithBlockingTimeoutPeriod(TimeSpan.FromMilliseconds(500))
29122913
.WithNotificationCenter(notificationCenter)
29132914
.Build();

OptimizelySDK/Config/HttpProjectConfigManager.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public class Builder
135135
private TimeSpan Period;
136136
private TimeSpan BlockingTimeoutSpan;
137137
private bool AutoUpdate = true;
138-
private bool StartByDefault;
138+
private bool StartByDefault = true;
139139
private NotificationCenter NotificationCenter;
140140

141141

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ This option enables user to provide a custom URL format to fetch the datafile.
172172

173173
##### Start by default
174174

175-
This option is used to specify whether to start the config manager on initialization.
175+
This option is used to specify whether to start the config manager on initialization or not. If no value is provided, by default it is true and will start polling datafile from remote immediately.
176176

177177
## Development
178178

0 commit comments

Comments
 (0)