Merge pull request #4468 from matrix-org/travis/rel-seshat-enable

Take encrypted message search out of labs for release
This commit is contained in:
J. Ryan Stinnett 2020-04-23 10:22:04 +01:00 committed by GitHub
commit 9b4172bf85
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 8 additions and 22 deletions

View file

@ -163,7 +163,7 @@ export default class EventIndexPanel extends React.Component {
); );
eventIndexingSettings = ( eventIndexingSettings = (
<div> <div className='mx_SettingsTab_subsectionText'>
{ {
_t( "Riot is missing some components required for securely " + _t( "Riot is missing some components required for securely " +
"caching encrypted messages locally. If you'd like to " + "caching encrypted messages locally. If you'd like to " +
@ -180,7 +180,7 @@ export default class EventIndexPanel extends React.Component {
); );
} else { } else {
eventIndexingSettings = ( eventIndexingSettings = (
<div> <div className='mx_SettingsTab_subsectionText'>
{ {
_t( "Riot can't securely cache encrypted messages locally " + _t( "Riot can't securely cache encrypted messages locally " +
"while running in a web browser. Use <riotLink>Riot Desktop</riotLink> " + "while running in a web browser. Use <riotLink>Riot Desktop</riotLink> " +

View file

@ -254,15 +254,12 @@ export default class SecurityUserSettingsTab extends React.Component {
</div> </div>
); );
let eventIndex; const eventIndex = (
if (SettingsStore.isFeatureEnabled("feature_event_indexing")) {
eventIndex = (
<div className="mx_SettingsTab_section"> <div className="mx_SettingsTab_section">
<span className="mx_SettingsTab_subheading">{_t("Message search")}</span> <span className="mx_SettingsTab_subheading">{_t("Message search")}</span>
<EventIndexPanel /> <EventIndexPanel />
</div> </div>
); );
}
// XXX: There's no such panel in the current cross-signing designs, but // XXX: There's no such panel in the current cross-signing designs, but
// it's useful to have for testing the feature. If there's no interest // it's useful to have for testing the feature. If there's no interest

View file

@ -400,7 +400,6 @@
"Show a presence dot next to DMs in the room list": "Show a presence dot next to DMs in the room list", "Show a presence dot next to DMs in the room list": "Show a presence dot next to DMs in the room list",
"Support adding custom themes": "Support adding custom themes", "Support adding custom themes": "Support adding custom themes",
"Enable cross-signing to verify per-user instead of per-session": "Enable cross-signing to verify per-user instead of per-session", "Enable cross-signing to verify per-user instead of per-session": "Enable cross-signing to verify per-user instead of per-session",
"Enable local event indexing and E2EE search (requires restart)": "Enable local event indexing and E2EE search (requires restart)",
"Show info about bridges in room settings": "Show info about bridges in room settings", "Show info about bridges in room settings": "Show info about bridges in room settings",
"Show padlocks on invite only rooms": "Show padlocks on invite only rooms", "Show padlocks on invite only rooms": "Show padlocks on invite only rooms",
"Enable Emoji suggestions while typing": "Enable Emoji suggestions while typing", "Enable Emoji suggestions while typing": "Enable Emoji suggestions while typing",

View file

@ -37,10 +37,6 @@ class EventIndexPeg {
* EventIndex was successfully initialized, false otherwise. * EventIndex was successfully initialized, false otherwise.
*/ */
async init() { async init() {
if (!SettingsStore.isFeatureEnabled("feature_event_indexing")) {
return false;
}
const indexManager = PlatformPeg.get().getEventIndexingManager(); const indexManager = PlatformPeg.get().getEventIndexingManager();
if (!indexManager) { if (!indexManager) {
console.log("EventIndex: Platform doesn't support event indexing, not initializing."); console.log("EventIndex: Platform doesn't support event indexing, not initializing.");

View file

@ -158,12 +158,6 @@ export const SETTINGS = {
supportedLevels: ['device', 'config'], // we shouldn't use LEVELS_FEATURE for non-features, so copy it here. supportedLevels: ['device', 'config'], // we shouldn't use LEVELS_FEATURE for non-features, so copy it here.
default: true, default: true,
}, },
"feature_event_indexing": {
isFeature: true,
supportedLevels: LEVELS_FEATURE,
displayName: _td("Enable local event indexing and E2EE search (requires restart)"),
default: false,
},
"feature_bridge_state": { "feature_bridge_state": {
isFeature: true, isFeature: true,
supportedLevels: LEVELS_FEATURE, supportedLevels: LEVELS_FEATURE,