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 #142

Closed
manuroe opened this issue May 21, 2019 · 4 comments
Closed

Support clear reactions in e2e rooms #142

manuroe opened this issue May 21, 2019 · 4 comments
Labels
A-Aggregations A-Reactions

Comments

@manuroe
Copy link
Member

manuroe commented May 21, 2019 •
edited
Loading

From riot-ios created by manuroe: element-hq/element-ios#2448

As explained by Ryan at #142 (comment)) for riot-web
@manuroe manuroe mentioned this issue May 21, 2019
Closed
@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

jryans commented Jun 26, 2019

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

@manuroe
Copy link
Member Author

manuroe commented Jul 2, 2019

Encrypted reactions will be supported later in phase:3

@manuroe manuroe added phase:3 and removed phase:2 labels Jul 2, 2019
@manuroe manuroe changed the title Support reactions in e2e rooms Support clear reactions in e2e rooms Jul 3, 2019
@manuroe
Copy link
Member Author

manuroe commented Jul 3, 2019

Just tested on RiotX. They are already sent in clear.

@manuroe manuroe closed this as completed Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Aggregations A-Reactions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jryans @manuroe