Page MenuHomePhabricator
Differential D6041

[web] Keyboard support for typeahead [1/13] - Moved getTypeaheadUserSuggestions to lib
ClosedPublic
Actions

Authored by • przemek on Dec 27 2022, 4:31 AM.
Tags
None
Referenced Files
F5111541: D6041.diff
Wed, Apr 2, 4:21 AM
Unknown Object (File)
Mon, Mar 31, 5:53 PM
Unknown Object (File)
Mon, Mar 17, 9:21 AM
Unknown Object (File)
Mon, Mar 17, 9:21 AM
Unknown Object (File)
Mon, Mar 17, 9:21 AM
Unknown Object (File)
Mon, Mar 17, 9:21 AM
Unknown Object (File)
Mon, Mar 17, 9:13 AM
Unknown Object (File)
Fri, Mar 7, 11:56 AM
View All Files
Subscribers
ashoat
atul
ginsu
inka
kamil
tomek

Details

Reviewers
inka
kamil
ginsu
tomek
Commits
rCOMM15354950eecb: [web] Keyboard support for typeahead [1/13] - Moved getTypeaheadUserSuggestions…
Summary

Moved function that will be common for web and native to lib

Test Plan

Typeahead without keyboard support works.
Final tests performed in last diffs.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• przemek created this revision.Dec 27 2022, 4:31 AM
Herald added subscribers: atul, ashoat. · View Herald TranscriptDec 27 2022, 4:31 AM
Harbormaster completed remote builds in B14774: Diff 20152.Dec 27 2022, 4:46 AM
• przemek requested review of this revision.Dec 27 2022, 4:46 AM
• przemek added a parent revision: D5819: [web] Add other <button> props to Button..Dec 27 2022, 5:30 AM
• przemek added a child revision: D6042: [web] Keyboard support for typeahead [2/13] - Moved typeahead-utils.js.
ginsu accepted this revision.Dec 27 2022, 3:38 PM
This revision is now accepted and ready to land.Dec 27 2022, 3:38 PM
• przemek added a reviewer: tomek.Dec 29 2022, 11:55 AM
This revision now requires review to proceed.Dec 29 2022, 11:55 AM
tomek accepted this revision.Dec 30 2022, 6:42 AM
This revision is now accepted and ready to land.Dec 30 2022, 6:42 AM
• przemek updated this revision to Diff 20852.Jan 12 2023, 1:54 AM

rebase

Harbormaster completed remote builds in B15290: Diff 20852.Jan 12 2023, 2:07 AM
Closed by commit rCOMM15354950eecb: [web] Keyboard support for typeahead [1/13] - Moved getTypeaheadUserSuggestions… (authored by • przemek). · Explain WhyJan 12 2023, 6:18 AM
This revision was automatically updated to reflect the committed changes.
• przemek added a commit: rCOMM15354950eecb: [web] Keyboard support for typeahead [1/13] - Moved getTypeaheadUserSuggestions….

Revision Contents
Changeset List

PathSize
lib/
shared/
21 lines
web/
chat/
2 lines
16 lines

Diff 20852

View Options

lib/shared/typeahead-utils.js

  • This file was added.
// @flow
import type { Account as OlmAccount } from '@commapp/olm';
import { ServerError } from 'lib/utils/errors.js';
import { dbQuery, SQL } from '../database/database.js';
import { fetchCallUpdateOlmAccount } from '../updaters/olm-account-updater.js';
import { createPickledOlmSession } from '../utils/olm-utils.js';
async function createOlmSession(
initialEncryptedMessage: string,
olmSessionType: 'content' | 'notifications',
cookieID: string,
): Promise<void> {
const callback = (account: OlmAccount, picklingKey: string) =>
createPickledOlmSession(account, picklingKey, initialEncryptedMessage);
let pickledOlmSession;
try {
pickledOlmSession = await fetchCallUpdateOlmAccount(
olmSessionType,
callback,
);
} catch (e) {
throw new ServerError('olm_session_creation_failure');
}
const isContent = olmSessionType === 'content';
// We match the native client behavior here where olm session is overwritten
// in case it is initialized twice for the same pair of identities
await dbQuery(
SQL`
INSERT INTO olm_sessions (cookie_id, is_content,
version, pickled_olm_session)
VALUES (${cookieID}, ${isContent}, 0, ${pickledOlmSession})
ON DUPLICATE KEY UPDATE
pickled_olm_session = ${pickledOlmSession}
`,
);
}
export { createOlmSession };
View Options

web/chat/typeahead-tooltip.react.js

  • This file was added.
// @flow
import type { Account as OlmAccount } from '@commapp/olm';
import { ServerError } from 'lib/utils/errors.js';
import { dbQuery, SQL } from '../database/database.js';
import { fetchCallUpdateOlmAccount } from '../updaters/olm-account-updater.js';
import { createPickledOlmSession } from '../utils/olm-utils.js';
async function createOlmSession(
initialEncryptedMessage: string,
olmSessionType: 'content' | 'notifications',
cookieID: string,
): Promise<void> {
const callback = (account: OlmAccount, picklingKey: string) =>
createPickledOlmSession(account, picklingKey, initialEncryptedMessage);
let pickledOlmSession;
try {
pickledOlmSession = await fetchCallUpdateOlmAccount(
olmSessionType,
callback,
);
} catch (e) {
throw new ServerError('olm_session_creation_failure');
}
const isContent = olmSessionType === 'content';
// We match the native client behavior here where olm session is overwritten
// in case it is initialized twice for the same pair of identities
await dbQuery(
SQL`
INSERT INTO olm_sessions (cookie_id, is_content,
version, pickled_olm_session)
VALUES (${cookieID}, ${isContent}, 0, ${pickledOlmSession})
ON DUPLICATE KEY UPDATE
pickled_olm_session = ${pickledOlmSession}
`,
);
}
export { createOlmSession };
View Options

web/chat/typeahead-utils.js

  • This file was added.
// @flow
import type { Account as OlmAccount } from '@commapp/olm';
import { ServerError } from 'lib/utils/errors.js';
import { dbQuery, SQL } from '../database/database.js';
import { fetchCallUpdateOlmAccount } from '../updaters/olm-account-updater.js';
import { createPickledOlmSession } from '../utils/olm-utils.js';
async function createOlmSession(
initialEncryptedMessage: string,
olmSessionType: 'content' | 'notifications',
cookieID: string,
): Promise<void> {
const callback = (account: OlmAccount, picklingKey: string) =>
createPickledOlmSession(account, picklingKey, initialEncryptedMessage);
let pickledOlmSession;
try {
pickledOlmSession = await fetchCallUpdateOlmAccount(
olmSessionType,
callback,
);
} catch (e) {
throw new ServerError('olm_session_creation_failure');
}
const isContent = olmSessionType === 'content';
// We match the native client behavior here where olm session is overwritten
// in case it is initialized twice for the same pair of identities
await dbQuery(
SQL`
INSERT INTO olm_sessions (cookie_id, is_content,
version, pickled_olm_session)
VALUES (${cookieID}, ${isContent}, 0, ${pickledOlmSession})
ON DUPLICATE KEY UPDATE
pickled_olm_session = ${pickledOlmSession}
`,
);
}
export { createOlmSession };