Skip to content
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

Add basic MAM support #1306

Merged
merged 11 commits into from
Apr 13, 2020
Merged

Add basic MAM support #1306

merged 11 commits into from
Apr 13, 2020

Conversation

jubalh
Copy link
Member

@jubalh jubalh commented Apr 8, 2020

No description provided.

Send a request (which we can't handle yet) :-)

Regards #660
MAM messages don't have a type nor a from.
If we detect a message without type let's log it and exit without
continuing to try to parse it.

Otherwise we go into _handle_chat() and crash on the no from.
Is usefult in many cases if we want cleaner code.
Hope this edit didn't break anything though ;-)
Later we will have several options.
Getting everything since last timestamp (if none everything at all).
Getting everything since today + configure time (1 week).

Should also have a reload all command like conversations once you
cleared the history.

All MAM messages should be written into sql db.
And then probably displayed from there so that regular history works
too.
Only when we start the conversation.
Not yet when we get messaged and a new window is opened.
Need to have sorting of messages in the window buffer then, I guess.
Also MAM IQ should only be send one time in such a case.

If MAM is enabled history from sql backend will not be shown.

`mam` in profrc enables experimental MAM.
Can change soon again. Don't rely on stuff in this stage ;)
g_date_time_add_days() actually creates a new one.
@jubalh jubalh marked this pull request as ready for review April 13, 2020 08:49
@jubalh
Copy link
Member Author

jubalh commented Apr 13, 2020

Only contains basic support.
More work is planned and needed.
Don't enable it yet if you don't know what you are doing. Only for devs.

@jubalh jubalh merged commit a800bfc into master Apr 13, 2020
@jubalh jubalh added this to the 0.9.0 milestone Apr 13, 2020
@jubalh jubalh deleted the feature/mam branch April 13, 2020 08:56
jubalh added a commit that referenced this pull request Jul 3, 2023
Started in #1306
for 0.9.0 by me.
We mentioned that it was experimental.

Later heavily improved in
#1724
for upcoming version by MarcoPolo-PasTonMolo.

MUC MAM is in development at:
#1862

Tracking issue: #660
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant