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.

Sign up for GitHub

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

Jump to bottom

Support clear reactions in e2e rooms #2448

Closed
manuroe opened this issue May 21, 2019 · 3 comments · Fixed by matrix-org/matrix-ios-sdk#687
Closed

Support clear reactions in e2e rooms #2448

manuroe opened this issue May 21, 2019 · 3 comments · Fixed by matrix-org/matrix-ios-sdk#687
Assignees
@manuroe
Labels
A-Aggregations A-Redaction feature

Comments

@manuroe
Copy link
Member

manuroe commented May 21, 2019 •
edited
Loading

As explained by Ryan at #2448 (comment) for riot-web
@manuroe manuroe mentioned this issue May 21, 2019
Closed
@manuroe
Copy link
Member Author

manuroe commented May 21, 2019

Kamino cloned this issue to vector-im/riotX-android

@manuroe manuroe mentioned this issue May 21, 2019
Closed
20 tasks
@manuroe manuroe added phase:1 and removed phase:2 labels May 24, 2019
@manuroe
Copy link
Member Author

manuroe commented Jun 11, 2019

It works in phase:1 but for main use cases. e2e will be officially supported in phase:2.

@manuroe manuroe added phase:2 and removed phase:1 labels Jun 11, 2019
@jryans
Copy link
Contributor

jryans commented Jun 26, 2019

Riot Web has changed to special case reactions so that they are sent without encryption to avoid notifications.

@manuroe manuroe removed the X-Blocked label Jul 2, 2019
@manuroe manuroe changed the title Support reactions in e2e rooms Support clear reactions in e2e rooms Jul 2, 2019
@manuroe manuroe self-assigned this Jul 3, 2019
manuroe added a commit to matrix-org/matrix-ios-sdk that referenced this issue Jul 3, 2019
@manuroe
Reactions: Send them in clear in e2e rooms for the moment
ffc6316 
element-hq/element-ios#2448
@manuroe manuroe mentioned this issue Jul 3, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@manuroe manuroe

Labels
A-Aggregations A-Redaction feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Reactions: Send them in clear in e2e rooms for the moment matrix-org/matrix-ios-sdk
2 participants
@jryans @manuroe