-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
XEP-0313 MAM Implementation #5
Comments
Conversations handles this pretty well in regards of de-duplication, synchronizing read status and so on.
|
IMO it will be conceptually right to implement logger in connection manager (because the logger needs to know more and more details to work properly, see also TelepathyIM/telepathy-spec#1)
My summary:
I tried to use MessageSent for |
So far i was working only with qmlmessages/commhistd client implementation so was thinking should be sufficient just to emit Carbons/MAM messages with proper timestamp and flags on the normal text channel and they will be consumed and stored to history. And to trigger sync on a) connection apparently b) active state (that was my PoC idea). However yes now deduplication problem comes into picture as each MAM sync will potentially re-emit certain messages. and with Carbons the duplication almost inevitable (unless we make the properties mutually exclusive). |
Nit: The issue title is correct, this is XEP-0313, not XEP-0363. |
fixed, thx. |
Carbons is good and well needed but there's tendency to shift towards MAM using it as subscription to pick up communication. Also it makes more sense to have it for mobile devices (sleep, wake, sync).
Any volunteers or perhaps even ready implementations? I have it on to-do list but not sure how fast i can approach it.
The text was updated successfully, but these errors were encountered: