Page MenuHomePhabricator

[lib] Fix currentAsOf
ClosedPublic

Authored by inka on Jan 16 2024, 6:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 7:41 AM
Unknown Object (File)
Fri, Oct 18, 2:36 PM
Unknown Object (File)
Fri, Oct 18, 2:36 PM
Unknown Object (File)
Fri, Oct 18, 2:35 PM
Unknown Object (File)
Fri, Oct 18, 6:55 AM
Unknown Object (File)
Wed, Oct 16, 1:17 AM
Unknown Object (File)
Wed, Oct 16, 1:10 AM
Unknown Object (File)
Sep 6 2024, 9:27 PM
Subscribers

Details

Summary

If the keyserver was not present in the message store previously, processedMessageStore.currentAsOf[keyserverID] will be undefined. And max from number and undefined is NaN

Test Plan

Tested that now when a user logs in to a new keyserver and some messages are sent from it, the value is not NaN

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 16 2024, 6:59 AM
Harbormaster failed remote builds in B25799: Diff 35668!
inka requested review of this revision.Jan 16 2024, 7:51 AM
This revision is now accepted and ready to land.Jan 16 2024, 6:58 PM
This revision was automatically updated to reflect the committed changes.