Page MenuHomePhabricator
Differential D10652

[lib] Fix currentAsOf
ClosedPublic
Actions

Authored by inka on Jan 16 2024, 6:50 AM.
Tags
None
Referenced Files
F3350144: D10652.id35702.diff
Fri, Nov 22, 8:56 PM
F3349134: D10652.diff
Fri, Nov 22, 5:20 PM
Unknown Object (File)
Fri, Nov 8, 1:06 AM
Unknown Object (File)
Mon, Nov 4, 7:41 AM
Unknown Object (File)
Oct 18 2024, 2:36 PM
Unknown Object (File)
Oct 18 2024, 2:36 PM
Unknown Object (File)
Oct 18 2024, 2:35 PM
Unknown Object (File)
Oct 18 2024, 6:55 AM
View All 60 Files
Subscribers
ashoat
tomek

Details

Reviewers
michal
kamil
ginsu
ashoat
Commits
rCOMMe5200cc68cad: [lib] Fix currentAsOf
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
Branch
inka/login_reducers
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka created this revision.Jan 16 2024, 6:50 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 16 2024, 6:50 AM
inka added a parent revision: D10650: [lib] Refactor logout in reduceEntryInfos.Jan 16 2024, 6:50 AM
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 added a child revision: D10653: lib] Refactor message reducer.Jan 16 2024, 7:43 AM
Harbormaster completed remote builds in B25799: Diff 35668.Jan 16 2024, 7:51 AM
inka requested review of this revision.Jan 16 2024, 7:51 AM
ashoat accepted this revision.Jan 16 2024, 6:58 PM
This revision is now accepted and ready to land.Jan 16 2024, 6:58 PM
inka removed a child revision: D10653: lib] Refactor message reducer.Jan 17 2024, 1:05 AM
inka added a child revision: D10653: lib] Refactor message reducer.
inka removed a parent revision: D10650: [lib] Refactor logout in reduceEntryInfos.
Closed by commit rCOMMe5200cc68cad: [lib] Fix currentAsOf (authored by inka). · Explain WhyJan 17 2024, 1:07 AM
This revision was automatically updated to reflect the committed changes.
inka added a commit: rCOMMe5200cc68cad: [lib] Fix currentAsOf.

Revision Contents
Changeset List

PathSize
lib/
reducers/
2 lines

Diff 35668

View Options

lib/reducers/message-reducer.js

Loading...