Skip to content

Commit bb3cf85

Browse files
authored
Merge pull request #94 from SenaxInc/copilot/fix-time-not-set-error
Handle Notecard "time not yet set" error during startup
2 parents 06286e4 + ded1321 commit bb3cf85

File tree

3 files changed

+28
-0
lines changed

3 files changed

+28
-0
lines changed

TankAlarm-112025-Client-BluesOpta/TankAlarm-112025-Client-BluesOpta.ino

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -676,6 +676,16 @@ static void syncTimeFromNotecard() {
676676
return;
677677
}
678678

679+
// Check for error response (e.g., "time is not yet set {no-time}")
680+
// This is normal during startup before Notecard syncs with cloud
681+
const char *err = JGetString(rsp, "err");
682+
if (err && strlen(err) > 0) {
683+
// Time not yet available - this is expected during startup
684+
// Will retry on next call - don't count this as a failure
685+
notecard.deleteResponse(rsp);
686+
return;
687+
}
688+
679689
double time = JGetNumber(rsp, "time");
680690
if (time > 0) {
681691
gLastSyncedEpoch = time;

TankAlarm-112025-Server-BluesOpta/TankAlarm-112025-Server-BluesOpta.ino

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2838,6 +2838,15 @@ static void ensureTimeSync() {
28382838
if (!rsp) {
28392839
return;
28402840
}
2841+
// Check for error response (e.g., "time is not yet set {no-time}")
2842+
// This is normal during startup before Notecard syncs with cloud
2843+
const char *err = JGetString(rsp, "err");
2844+
if (err && strlen(err) > 0) {
2845+
// Time not yet available - this is expected during startup
2846+
// Will retry on next call
2847+
notecard.deleteResponse(rsp);
2848+
return;
2849+
}
28412850
double time = JGetNumber(rsp, "time");
28422851
if (time > 0) {
28432852
gLastSyncedEpoch = time;

TankAlarm-112025-Viewer-BluesOpta/TankAlarm-112025-Viewer-BluesOpta.ino

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -570,6 +570,15 @@ static void ensureTimeSync() {
570570
if (!rsp) {
571571
return;
572572
}
573+
// Check for error response (e.g., "time is not yet set {no-time}")
574+
// This is normal during startup before Notecard syncs with cloud
575+
const char *err = JGetString(rsp, "err");
576+
if (err && strlen(err) > 0) {
577+
// Time not yet available - this is expected during startup
578+
// Will retry on next call
579+
notecard.deleteResponse(rsp);
580+
return;
581+
}
573582
double time = JGetNumber(rsp, "time");
574583
if (time > 0) {
575584
gLastSyncedEpoch = time;

0 commit comments

Comments
 (0)