Page MenuHomePhabricator
Feed All Stories

Jul 6 2024

ashoat requested review of D12678: [native] Don't bother with keyserver auth in BackgroundIdentityLoginHandler.
Jul 6 2024, 2:56 PM
ashoat added a reverting change for D12674: [lib] Fix missing usernames: D12680: [lib] Fix crash when pressing message from anonymous author.
Jul 6 2024, 2:42 PM

Jul 5 2024

ashoat accepted D12675: [native] update copy for thread notification settings.
Jul 5 2024, 1:31 PM
Harbormaster failed remote builds in B30173: Diff 42084 for D12675: [native] update copy for thread notification settings!
Jul 5 2024, 1:24 PM
ginsu updated the diff for D12675: [native] update copy for thread notification settings.

update

Jul 5 2024, 12:59 PM
ginsu planned changes to D12675: [native] update copy for thread notification settings.

Found a bug with saving the thread notif settings

Jul 5 2024, 12:45 PM
marcin requested review of D12676: Migrate notifications crypto account to MMKV on native.
Jul 5 2024, 11:37 AM
ginsu requested review of D12675: [native] update copy for thread notification settings.
Jul 5 2024, 11:21 AM
ashoat committed rCOMM89dc3aef75e2: Revert "[lib] Temporarily flip usingCommServicesAccessToken to false" (authored by ashoat).
Revert "[lib] Temporarily flip usingCommServicesAccessToken to false"
Jul 5 2024, 9:58 AM
ashoat added a reverting change for rCOMMf0ddbc7fb5b8: [lib] Temporarily flip usingCommServicesAccessToken to false: rCOMM89dc3aef75e2: Revert "[lib] Temporarily flip usingCommServicesAccessToken to false".
Jul 5 2024, 9:58 AM
ashoat added a reverting change for rCOMM7be353f6f870: [native] Temporary changes for staff release: rCOMM200140b0871c: Revert "[native] Temporary changes for staff release".
Jul 5 2024, 9:58 AM
ashoat committed rCOMM200140b0871c: Revert "[native] Temporary changes for staff release" (authored by ashoat).
Revert "[native] Temporary changes for staff release"
Jul 5 2024, 9:58 AM
ashoat committed rCOMMd83c2c61c5c6: [native] codeVersion -> 366 (authored by ashoat).
[native] codeVersion -> 366
Jul 5 2024, 9:58 AM
ashoat committed rCOMM7be353f6f870: [native] Temporary changes for staff release (authored by ashoat).
[native] Temporary changes for staff release
Jul 5 2024, 9:58 AM
ashoat committed rCOMM1f0fa1cf6df7: [native] codeVersion -> 365 (authored by ashoat).
[native] codeVersion -> 365
Jul 5 2024, 9:58 AM
ashoat committed rCOMM7532381c210f: [web][keyserver] codeVersion -> 95 (authored by ashoat).
[web][keyserver] codeVersion -> 95
Jul 5 2024, 9:58 AM
ashoat committed rCOMMf0ddbc7fb5b8: [lib] Temporarily flip usingCommServicesAccessToken to false (authored by ashoat).
[lib] Temporarily flip usingCommServicesAccessToken to false
Jul 5 2024, 9:58 AM
inka closed D12674: [lib] Fix missing usernames.
Jul 5 2024, 9:27 AM
inka committed rCOMM4f33a2424c1a: [lib] Fix missing usernames (authored by inka).
[lib] Fix missing usernames
Jul 5 2024, 9:27 AM
tomek accepted D12674: [lib] Fix missing usernames.
Jul 5 2024, 9:12 AM
tomek published D12674: [lib] Fix missing usernames for review.
Jul 5 2024, 9:11 AM
ashoat added inline comments to D12658: [lib] DMOperationSpec for CREATE_THREAD operation.
Jul 5 2024, 8:31 AM
tomek accepted D12658: [lib] DMOperationSpec for CREATE_THREAD operation.
Jul 5 2024, 5:09 AM
tomek added a comment to D12422: [web][native] Display both thread and community names in the modal.
In D12422#355471, @inka wrote:

"within" is not centered on web (vertically), can we fix this?

Jul 5 2024, 4:36 AM
inka planned changes to D12667: [web] Fix creating subchannels in chats in GENESIS.

On Linear, you asked whether we should check the parent or the top ancestor under GENESIS for permissions. I suggested that the top ancestor made more sense, but I probably care more about consistency with native. What do we do there?

Jul 5 2024, 2:48 AM
inka accepted D12657: [lib] Permissions for thick threads.
Jul 5 2024, 1:49 AM

Jul 4 2024

ashoat committed rCOMMff18eb3b1024: Revert "[native] Temporary changes for staff release" (authored by ashoat).
Revert "[native] Temporary changes for staff release"
Jul 4 2024, 12:16 PM
ashoat committed rCOMM24c1ee40ddf4: [native] Temporary changes for staff release (authored by ashoat).
[native] Temporary changes for staff release
Jul 4 2024, 12:16 PM
ashoat committed rCOMMc3a8411053b5: [native] codeVersion -> 364 (authored by ashoat).
[native] codeVersion -> 364
Jul 4 2024, 12:16 PM
ashoat committed rCOMM9531e5e257ce: Revert "[lib] Temporarily flip usingCommServicesAccessToken to false" (authored by ashoat).
Revert "[lib] Temporarily flip usingCommServicesAccessToken to false"
Jul 4 2024, 12:16 PM
ashoat committed rCOMM3934a0450ac6: [native] codeVersion -> 363 (authored by ashoat).
[native] codeVersion -> 363
Jul 4 2024, 12:16 PM
ashoat committed rCOMMf94ca9567835: [web][keyserver] codeVersion -> 94 (authored by ashoat).
[web][keyserver] codeVersion -> 94
Jul 4 2024, 12:16 PM
ashoat committed rCOMM96fb14a8477c: [lib] Temporarily flip usingCommServicesAccessToken to false (authored by ashoat).
[lib] Temporarily flip usingCommServicesAccessToken to false
Jul 4 2024, 12:16 PM
ashoat accepted D12667: [web] Fix creating subchannels in chats in GENESIS.

On Linear, you asked whether we should check the parent or the top ancestor under GENESIS for permissions. I suggested that the top ancestor made more sense, but I probably care more about consistency with native. What do we do there?

Jul 4 2024, 11:31 AM
ashoat added inline comments to D12650: Update extractKeyserverIDFromID so that it returns null for DMs.
Jul 4 2024, 11:28 AM
ashoat added inline comments to D12638: [lib] Send primary device logout message to secondary devices.
Jul 4 2024, 11:23 AM
ginsu closed D12666: [native] deprecate ThreadSettingsHomeNotifs.
Jul 4 2024, 11:21 AM
ginsu closed D12665: [native] update ThreadSettingsPushNotifs to use new thread notifs settings experience.
Jul 4 2024, 11:21 AM
ginsu committed rCOMMd78ef501ea74: [native] update ThreadSettingsPushNotifs to use new thread notifs settings… (authored by ginsu).
[native] update ThreadSettingsPushNotifs to use new thread notifs settings…
Jul 4 2024, 11:21 AM
ginsu committed rCOMM686c0d8f3f29: [native] deprecate ThreadSettingsHomeNotifs (authored by ginsu).
[native] deprecate ThreadSettingsHomeNotifs
Jul 4 2024, 11:21 AM
ginsu closed D12663: [native] update thread notif settings ui to handle case when parent thread is in background.
Jul 4 2024, 11:21 AM
ginsu closed D12664: [lib/native/web] update copy BADGE_COUNT => NOTIF_COUNT.
Jul 4 2024, 11:21 AM
ginsu committed rCOMMe60d77284b98: [native] update thread notif settings ui to handle case when parent thread is… (authored by ginsu).
[native] update thread notif settings ui to handle case when parent thread is…
Jul 4 2024, 11:21 AM
ginsu committed rCOMM17786b62ab07: [lib/native/web] update copy BADGE_COUNT => NOTIF_COUNT (authored by ginsu).
[lib/native/web] update copy BADGE_COUNT => NOTIF_COUNT
Jul 4 2024, 11:21 AM
ginsu closed D12662: [native] introduce thread notif settings notice text.
Jul 4 2024, 11:21 AM
ginsu closed D12648: [native] introduce NotificationDescription.
Jul 4 2024, 11:21 AM
ginsu committed rCOMMeb5b3d8405ec: [native] introduce NotificationDescription (authored by ginsu).
[native] introduce NotificationDescription
Jul 4 2024, 11:21 AM
ginsu committed rCOMM0e0eeebb12fa: [native] introduce thread notif settings notice text (authored by ginsu).
[native] introduce thread notif settings notice text
Jul 4 2024, 11:21 AM
ginsu closed D12647: [native] introduce barebones UI for ThreadSettingsNotifications.
Jul 4 2024, 11:21 AM
ginsu closed D12661: [lib/native/web] factor out remaining shared logic + copy for thread notification settings.
Jul 4 2024, 11:21 AM
ginsu committed rCOMM769fcab17eac: [native] introduce barebones UI for ThreadSettingsNotifications (authored by ginsu).
[native] introduce barebones UI for ThreadSettingsNotifications
Jul 4 2024, 11:21 AM
ginsu committed rCOMMd805cb6dbfba: [lib/native/web] factor out remaining shared logic + copy for thread… (authored by ginsu).
[lib/native/web] factor out remaining shared logic + copy for thread…
Jul 4 2024, 11:21 AM
ginsu closed D12654: [native] update EnumSettingsOption UI.
Jul 4 2024, 11:21 AM
ginsu closed D12653: [native] introduce disabled prop for EnumSettingsOption.
Jul 4 2024, 11:21 AM
ginsu committed rCOMM35a34d3e2ca4: [native] update EnumSettingsOption UI (authored by ginsu).
[native] update EnumSettingsOption UI
Jul 4 2024, 11:21 AM
ginsu committed rCOMMa9358077a210: [native] introduce disabled prop for EnumSettingsOption (authored by ginsu).
[native] introduce disabled prop for EnumSettingsOption
Jul 4 2024, 11:21 AM
ginsu closed D12645: [native] extend icon prop in EnumSettingsOption to handle illustration components.
Jul 4 2024, 11:21 AM
ginsu committed rCOMM5795c5998829: [native] extend icon prop in EnumSettingsOption to handle illustration… (authored by ginsu).
[native] extend icon prop in EnumSettingsOption to handle illustration…
Jul 4 2024, 11:21 AM
ginsu closed D12643: [native] introduce type prop to EnumSettingsOption component.
Jul 4 2024, 11:21 AM
ginsu committed rCOMM8ec15fd60aa1: [native] introduce type prop to EnumSettingsOption component (authored by ginsu).
[native] introduce type prop to EnumSettingsOption component
Jul 4 2024, 11:21 AM
ginsu closed D12642: [native] introduce nav logic for ThreadSettingsNotifications.
Jul 4 2024, 11:21 AM
ginsu committed rCOMM4df530f032dc: [native] introduce nav logic for ThreadSettingsNotifications (authored by ginsu).
[native] introduce nav logic for ThreadSettingsNotifications
Jul 4 2024, 11:21 AM
ginsu updated the diff for D12666: [native] deprecate ThreadSettingsHomeNotifs.

rebae before landing

Jul 4 2024, 11:16 AM
ashoat closed D12659: [lib] Fix accidental variable naming from D12495.
Jul 4 2024, 11:15 AM
ashoat committed rCOMM1b3dff868ac3: [lib] Fix accidental variable naming from D12495 (authored by ashoat).
[lib] Fix accidental variable naming from D12495
Jul 4 2024, 11:15 AM
ashoat closed D12652: [lib] Introduce DMOperations.
Jul 4 2024, 11:15 AM
ashoat closed D12655: [lib] DMOperationSpec for SEND_TEXT_MESSAGE operation.
Jul 4 2024, 11:15 AM
ashoat closed D12656: [lib] Make it possible to skip some thread/message properties.
Jul 4 2024, 11:15 AM
ashoat closed D12660: [keyserver] Update ServerThreadInfo to only cover ThinThreadType.
Jul 4 2024, 11:15 AM
ashoat committed rCOMMbdece8406e42: [lib] Introduce DMOperations (authored by ashoat).
[lib] Introduce DMOperations
Jul 4 2024, 11:15 AM
ashoat committed rCOMM23cf032f052c: [lib] DMOperationSpec for SEND_TEXT_MESSAGE operation (authored by ashoat).
[lib] DMOperationSpec for SEND_TEXT_MESSAGE operation
Jul 4 2024, 11:15 AM
ashoat committed rCOMMa761d91e9ae2: [keyserver] Update ServerThreadInfo to only cover ThinThreadType (authored by ashoat).
[keyserver] Update ServerThreadInfo to only cover ThinThreadType
Jul 4 2024, 11:15 AM
ashoat committed rCOMM4f9c9876a478: [lib] Make it possible to skip some thread/message properties (authored by ashoat).
[lib] Make it possible to skip some thread/message properties
Jul 4 2024, 11:15 AM
ginsu updated the diff for D12665: [native] update ThreadSettingsPushNotifs to use new thread notifs settings experience.

address comments + rebase before landing

Jul 4 2024, 11:15 AM
ashoat added inline comments to D12658: [lib] DMOperationSpec for CREATE_THREAD operation.
Jul 4 2024, 11:12 AM
ginsu updated the diff for D12664: [lib/native/web] update copy BADGE_COUNT => NOTIF_COUNT.

address comments + rebase before landing

Jul 4 2024, 11:11 AM
ashoat updated the diff for D12658: [lib] DMOperationSpec for CREATE_THREAD operation.
  1. Actually pass sourceMessageAuthorID in based on sourceMessage
  2. Decided to use a UUID instead of 'ignored'. I'm pretty sure it's not used on the client anywhere, but figured it's better to avoid any risks in case it affects anything, given our tight timeline for launch
Jul 4 2024, 11:09 AM
ashoat updated the diff for D12655: [lib] DMOperationSpec for SEND_TEXT_MESSAGE operation.

Decided to use a UUID instead of 'ignored'. I'm pretty sure it's not used on the client anywhere, but figured it's better to avoid any risks in case it affects anything, given our tight timeline for launch

Jul 4 2024, 11:06 AM
ginsu updated the diff for D12663: [native] update thread notif settings ui to handle case when parent thread is in background.

rebase before landing

Jul 4 2024, 11:04 AM
ginsu updated the diff for D12662: [native] introduce thread notif settings notice text.

rebase before landing

Jul 4 2024, 11:03 AM
ginsu updated the diff for D12662: [native] introduce thread notif settings notice text.

rebase before landing

Jul 4 2024, 11:02 AM
ginsu updated the diff for D12648: [native] introduce NotificationDescription.

rebase before landing

Jul 4 2024, 11:01 AM
ashoat closed D12641: [keyserver][lib][native] Continue storing password hash in authoritative keyserver MariaDB.
Jul 4 2024, 11:00 AM
ashoat committed rCOMM51a51ba64cc9: [keyserver][lib][native] Continue storing password hash in authoritative… (authored by ashoat).
[keyserver][lib][native] Continue storing password hash in authoritative…
Jul 4 2024, 11:00 AM
ginsu updated the diff for D12647: [native] introduce barebones UI for ThreadSettingsNotifications.

rebase before landing

Jul 4 2024, 10:59 AM
ginsu updated the diff for D12661: [lib/native/web] factor out remaining shared logic + copy for thread notification settings.

rebase before landing

Jul 4 2024, 10:58 AM
ginsu attached a referenced file: F2189691: Screen Recording 2024-07-04 at 1.56.02 PM.mov.
Jul 4 2024, 10:57 AM
ginsu updated the test plan for D12654: [native] update EnumSettingsOption UI.
Jul 4 2024, 10:57 AM
ginsu updated the diff for D12654: [native] update EnumSettingsOption UI.

address comments + rebase before landing

Jul 4 2024, 10:56 AM
ginsu updated the diff for D12653: [native] introduce disabled prop for EnumSettingsOption.

rebase before landing

Jul 4 2024, 10:49 AM
ginsu updated the diff for D12645: [native] extend icon prop in EnumSettingsOption to handle illustration components.

rebase before landing

Jul 4 2024, 10:48 AM
ginsu updated the diff for D12643: [native] introduce type prop to EnumSettingsOption component.

rebase before landing

Jul 4 2024, 10:47 AM
ginsu updated the diff for D12642: [native] introduce nav logic for ThreadSettingsNotifications.

rebase before landing

Jul 4 2024, 10:47 AM
ashoat added a comment to D12657: [lib] Permissions for thick threads.
In D12657#358709, @inka wrote:

Should we be able to create a subchannel of a thick thread?

Jul 4 2024, 10:44 AM
marcin added a comment to D12673: Utilities to idempotently create content and notif sessions.

I am afraid that there is a vulnerability in this diff. Basically it is possible that two createOlmSessionWithPeer promises will execute concurrently. This might lead to corrupted state. In order to mitigate that I think it is best expose this method as a context. The context would internally keep a map like [(userID, deviceID)] : Promise mapping from particular deviceID and userID to currently executing promise (if any).

Jul 4 2024, 9:56 AM
marcin requested review of D12673: Utilities to idempotently create content and notif sessions.
Jul 4 2024, 9:54 AM
marcin requested review of D12672: Implement utility functions to check if we have content or notif session with device.
Jul 4 2024, 9:50 AM
marcin updated the test plan for D12668: Implement peer notifications session creation as outbound on native.
Jul 4 2024, 9:44 AM
marcin updated the test plan for D12669: Implement peer notifications session creation as outbound on web and add notifs outbound session creator to OlmAPI.
Jul 4 2024, 9:44 AM
marcin updated the test plan for D12670: Implement notification encryption on native.
Jul 4 2024, 9:44 AM