From b96c9f45344fc79a0c4cf12e52dcbe9f9cc421e1 Mon Sep 17 00:00:00 2001 From: Dima Gerasimov Date: Thu, 2 Jun 2022 13:22:42 +0100 Subject: [PATCH] fbmessenger: use both id and timestamp for merging --- my/fbmessenger/common.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/my/fbmessenger/common.py b/my/fbmessenger/common.py index 748b9d9..a6549d5 100644 --- a/my/fbmessenger/common.py +++ b/my/fbmessenger/common.py @@ -39,5 +39,7 @@ def _merge_messages(*sources: Iterator[Res[Message]]) -> Iterator[Res[Message]]: if isinstance(r, Exception): return str(r) else: - return r.id + # use both just in case, would be easier to spot tz issues + # similar to twitter, might make sense to generify/document as a pattern + return (r.id, r.dt) yield from unique_everseen(chain(*sources), key=key)