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

Show system message for OMEMO status in the conversation #126

Closed
wants to merge 0 commits into from
Closed

Show system message for OMEMO status in the conversation #126

wants to merge 0 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 8, 2018

Fix issue #125

@ghost
Copy link
Author

ghost commented Oct 8, 2018

Result of this PR: when user open a window for IM conversation:

  • If OMEMO enabled, there will be a system message "OMEMO encrypt conversation: Enabled"
  • If OMEMO available, there will have a system message "OMEMO encrypt conversation: Available"

@gkdr
Copy link
Owner

gkdr commented Oct 14, 2018

Thanks, I'll have a look. I'd probably rather add it as an option in the new options dialog though.

@ghost
Copy link
Author

ghost commented Oct 18, 2018

I saw the new options interface in dev. I could help to add this as a option if I have time to work on it.

@gkdr
Copy link
Owner

gkdr commented Oct 20, 2018

I was going to suggest adding it to the preferences.

@ghost ghost closed this Oct 22, 2018
@ghost ghost deleted the dev branch October 22, 2018 01:37
@ghost
Copy link
Author

ghost commented Oct 22, 2018

The github has problem, I can not push to my forked repository. I can not do any operation to it. I tried to deleted the fork , but it is still appeared in my profile and could be visited. It looks like turned to a "read-only" status. I will try to wait for somebody fix it, and probably create a new pull request for this feature after it recoverd.

This pull request was closed.
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.

1 participant