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

[#156864336] Feature reload mam #58

Merged
merged 14 commits into from
Apr 27, 2018
Merged

[#156864336] Feature reload mam #58

merged 14 commits into from
Apr 27, 2018

Conversation

mpope9
Copy link
Contributor

@mpope9 mpope9 commented Apr 27, 2018

archive table is queried upon muc_room_initialization and the archived messages are loaded into the muc room history. Right now querying the last 50 messages as a hard coded value.

@aghchan
@MattFoley

Copy link
Contributor

@aghchan aghchan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

other than the nit, 👍

src/mod_mam.erl Outdated
@@ -37,7 +37,9 @@
remove_user/2, remove_room/3, mod_opt_type/1, muc_process_iq/2,
muc_filter_message/3, message_is_archived/3, delete_old_messages/2,
get_commands_spec/0, msg_to_el/4, get_room_config/4, set_room_option/3,
offline_message/1, export/1]).
offline_message/1, send/4, export/1]).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra export for send here unused

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

Copy link

@MattFoley MattFoley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mpope9 mpope9 merged commit d71bf18 into master Apr 27, 2018
@mpope9 mpope9 deleted the feature-reload-mam branch April 27, 2018 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants