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

Enable seshat by default #13262

Closed
dbkr opened this issue Apr 18, 2020 · 1 comment · Fixed by matrix-org/matrix-react-sdk#4467
Closed

Enable seshat by default #13262

dbkr opened this issue Apr 18, 2020 · 1 comment · Fixed by matrix-org/matrix-react-sdk#4467
Assignees
Labels
A-E2EE-Cross-Signing A-Indexing Indexing messages via Seshat

Comments

@dbkr
Copy link
Member

dbkr commented Apr 18, 2020

It's on in the release config.json but we should also enable it by default.

@jryans
Copy link
Collaborator

jryans commented Apr 22, 2020

For this one, I believe we can safely enable by removing the flag. The feature itself won't active without the native components, which only exist in the 1.6 series (for official builds).

@turt2live turt2live self-assigned this Apr 22, 2020
turt2live added a commit to matrix-org/matrix-react-sdk that referenced this issue Apr 22, 2020
Fixes element-hq/element-web#13262

This is part of the cross-signing shipping master plan. Known issues relating to this feature are:
* element-hq/element-web#12896
* https://github.com/vector-im/riot-web/issues/12385
* element-hq/element-web#11831
* element-hq/element-web#11155

In theory, these are issues we're comfortable with shipping as we're already enabling it by default. This just makes it easier on everyone by removing the flag (making it still enabled by default).
turt2live added a commit to matrix-org/matrix-react-sdk that referenced this issue Apr 22, 2020
Fixes element-hq/element-web#13262

This is part of the cross-signing shipping master plan. Known issues relating to this feature are:
* element-hq/element-web#12896
* https://github.com/vector-im/riot-web/issues/12385
* element-hq/element-web#11831
* element-hq/element-web#11155

In theory, these are issues we're comfortable with shipping as we're already enabling it by default. This just makes it easier on everyone by removing the flag (making it still enabled by default).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE-Cross-Signing A-Indexing Indexing messages via Seshat
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants