-
Notifications
You must be signed in to change notification settings - Fork 604
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 support for mattermost threading #627
Conversation
Code Climate has analyzed commit 3e39c88 and detected 0 issues on this pull request. View more on Code Climate. |
@patcon thanks for the initiative :-) I'll test asap on an instance |
Almost works. You need to actually set the ParentID though :) Add matterbridge/bridge/mattermost/handlers.go Lines 101 to 108 in a2754f1
|
@patcon are you going to fix this PR with the mattermost issue or should I open a new one for this? |
487572a
to
426aa33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found some fixes!
P.S. share your ideas, feedbacks or issues with us at https://github.com/fixmie/feedback (this message will be removed after the beta stage).
5af2324
to
904c72e
Compare
This is untested, as I don't have access to a mattermost instance.