Page MenuHomePhabricator
Feed Advanced Search

Apr 17 2024

tomek accepted D11666: [native] Rename get/setCurrentUserID in CommCoreModule.

Should we regenerate a wasm file?

Apr 17 2024, 4:32 AM

Apr 16 2024

tomek updated the test plan for D11649: [web] Reassign the notifs session.
Apr 16 2024, 9:15 AM
tomek added a comment to D11649: [web] Reassign the notifs session.

I would amend to the test plan some steps that try to recreate race condition when we log in multiple pages at the same time. We should expect that after logging in to all pages we can see multiple notifs sessions in IndexedDB but when the notifs is received we can decrypt it and it removes all but one notif session.

Apr 16 2024, 9:14 AM
tomek updated the diff for D11649: [web] Reassign the notifs session.

Fix reassignment logic

Apr 16 2024, 9:14 AM
tomek added inline comments to D11648: [web] Create a function that reassigns notifs session.
Apr 16 2024, 8:45 AM
tomek updated the diff for D11648: [web] Create a function that reassigns notifs session.

Rebase

Apr 16 2024, 8:45 AM
tomek updated the diff for D11647: [web] Extract a function that determines where to store notifs session.

Rename

Apr 16 2024, 8:41 AM
tomek added inline comments to D11645: Force already logged in ETH users to create and sign backup message.
Apr 16 2024, 4:11 AM
tomek accepted D11645: Force already logged in ETH users to create and sign backup message.
Apr 16 2024, 3:27 AM

Apr 15 2024

tomek accepted D11663: [native] Stop using ReanimatedJSIModulePackage on Android.
Apr 15 2024, 7:28 AM
tomek added a comment to D11643: [web] Show a popup when client_version_unsupported.

Can't we simply reload the page when the client version is unsupported?

Apr 15 2024, 4:22 AM
tomek accepted D11659: [native][android] Move database and MMKV initialization to CommCoreJSIModulePackage.

Why moving this code to CommHybrid fixes the issue?

I believe because it executes later. More details here. My thinking was that it's safe to call from CommHybrid, since it's called from CommCoreJSIModulePackage.getJSIModules.

Since by that point other JSI libraries (presuamably including Expo libraries) seem ready for setup, I figured it was safe to call SecureStore from that location.

Apr 15 2024, 4:16 AM
tomek accepted D11660: [native] Recreate CommMMKV keys if they can't be loaded.
Apr 15 2024, 1:58 AM
tomek added a comment to D11659: [native][android] Move database and MMKV initialization to CommCoreJSIModulePackage.

Why moving this code to CommHybrid fixes the issue?

Apr 15 2024, 1:56 AM
tomek accepted D11653: [lib] Update `UPDATE_RELATIONSHIP.rawMessageInfoFromClientDB` to handle `farcaster_mutual`.
Apr 15 2024, 1:38 AM
tomek added inline comments to D11651: [lib] Include `[creator/target]FID` in `messageContentForServerDB` when update relationship op is `farcaster_mutual`.
Apr 15 2024, 1:37 AM
tomek accepted D11652: [lib] Update `UPDATE_RELATIONSHIP.rawMessageInfoFromServerDB` to handle `farcaster_mutual`.
Apr 15 2024, 1:37 AM
tomek accepted D11651: [lib] Include `[creator/target]FID` in `messageContentForServerDB` when update relationship op is `farcaster_mutual`.
Apr 15 2024, 1:35 AM
tomek accepted D11650: [lib] Split `UpdateRelationshipMessageInfo` into `Update[Traditional/Farcaster]RelationshipMessageInfo`.
Apr 15 2024, 1:24 AM
tomek accepted D11632: [lib][web][native] Pass preRequestUserState into invalidSessionRecovery.
Apr 15 2024, 1:21 AM
tomek requested changes to D11645: Force already logged in ETH users to create and sign backup message.
Apr 15 2024, 1:08 AM

Apr 12 2024

tomek requested review of D11649: [web] Reassign the notifs session.
Apr 12 2024, 11:51 AM
tomek requested review of D11648: [web] Create a function that reassigns notifs session.
Apr 12 2024, 11:36 AM
tomek requested review of D11647: [web] Extract a function that determines where to store notifs session.
Apr 12 2024, 11:26 AM
tomek accepted D11642: [lib] Split `rawUpdateRelationshipMessageInfoValidator` into `rawUpdate[Traditional/Farcaster]RelationshipMessageInfoValidator`.
Apr 12 2024, 2:41 AM
tomek accepted D11641: [lib] Split `UpdateRelationshipMessageData` into `UpdateTraditionalMessageData` and `UpdateFarcasterMessageData`.
Apr 12 2024, 2:39 AM

Apr 11 2024

tomek added a comment to D11624: [lib] Fix `farcasterRelationshipRequestValidator`.

Do we need a dedicated validator for UpdateRelationshipMessage?

Apr 11 2024, 8:27 AM
tomek accepted D11624: [lib] Fix `farcasterRelationshipRequestValidator`.
Apr 11 2024, 8:27 AM
tomek accepted D11612: [lib] Update `updateRelationshipInputValidator` to be union of `[traditional/farcaster]RelationshipRequestValidator`.
Apr 11 2024, 8:26 AM
tomek accepted D11611: [keyserver] Update `updateRelationshipsResponder` to handle `RelationshipRequest`.
Apr 11 2024, 8:24 AM
tomek closed D11573: [native] Clear auth metadata on logout.
Apr 11 2024, 5:38 AM
tomek committed rCOMM77893eed7c78: [native] Clear auth metadata on logout (authored by tomek).
[native] Clear auth metadata on logout
Apr 11 2024, 5:38 AM
tomek closed D11533: [web][native] Reset the state iff current user ID changes.
Apr 11 2024, 5:38 AM
tomek closed D11534: [web][native] Extract the common logic determining when to clear the data.
Apr 11 2024, 5:38 AM
tomek committed rCOMMa384021d4934: [web][native] Reset the state iff current user ID changes (authored by tomek).
[web][native] Reset the state iff current user ID changes
Apr 11 2024, 5:38 AM
tomek committed rCOMM505976095c33: [web][native] Extract the common logic determining when to clear the data (authored by tomek).
[web][native] Extract the common logic determining when to clear the data
Apr 11 2024, 5:38 AM
tomek closed D11532: [web][native] Refactor reset state logic in reducers.
Apr 11 2024, 5:38 AM
tomek committed rCOMM30b805a54fd1: [web][native] Refactor reset state logic in reducers (authored by tomek).
[web][native] Refactor reset state logic in reducers
Apr 11 2024, 5:38 AM
tomek updated the diff for D11573: [native] Clear auth metadata on logout.

Rename

Apr 11 2024, 5:17 AM
tomek updated the diff for D11533: [web][native] Reset the state iff current user ID changes.

Update the comment

Apr 11 2024, 5:10 AM
tomek accepted D11610: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster` action.
Apr 11 2024, 2:24 AM

Apr 10 2024

tomek accepted D11619: [lib] Add client_version_unsupported connectionIssue.
Apr 10 2024, 6:53 AM
tomek accepted D11609: [lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action.
Apr 10 2024, 6:51 AM
tomek accepted D11607: [keyserver] Update `createPersonalThreads` to support both `[Traditional/Farcaster]RelationshipRequest`.
Apr 10 2024, 6:47 AM
tomek accepted D11606: [lib] Update `FarcasterDataHandler` to construct `FarcasterRelationshipRequest` with `userIDsToFID`.
NOTE: The request constructed on the client will currently fail. We need to update both validators and responder on keyserver to support this new format.

A safer ordering might be to first introduce the support for a new format on the keyserver, gated behind code version, and then modify the format on the client.

Apr 10 2024, 6:43 AM

Apr 9 2024

tomek abandoned D6688: [keyserver] Refactoring `redisPublish` function to `messageBrokerPublish`.

Not sure if we still need this - the Linear issue has been canceled for a long time.

Apr 9 2024, 10:58 AM
tomek commandeered D6688: [keyserver] Refactoring `redisPublish` function to `messageBrokerPublish`.
Apr 9 2024, 10:57 AM
tomek abandoned D4749: [services] Tunnelbroker - Add timeout for a pop message waiting in AmqpManager tests.

We don't have the tests anymore and the service is now written in Rust.

Apr 9 2024, 10:56 AM
tomek commandeered D4749: [services] Tunnelbroker - Add timeout for a pop message waiting in AmqpManager tests.
Apr 9 2024, 10:55 AM
tomek added inline comments to D11573: [native] Clear auth metadata on logout.
Apr 9 2024, 6:14 AM
tomek updated the diff for D11573: [native] Clear auth metadata on logout.

Delete empty string checks. Move clearing the metadata to clearSensitiveData. Stop running secure store functions on the DB thread.

Apr 9 2024, 6:14 AM
tomek updated the diff for D11534: [web][native] Extract the common logic determining when to clear the data.

Delete unnecessary conditions

Apr 9 2024, 5:05 AM
tomek added inline comments to D11533: [web][native] Reset the state iff current user ID changes.
Apr 9 2024, 4:54 AM
tomek updated the diff for D11533: [web][native] Reset the state iff current user ID changes.

Handle null user and update the comment

Apr 9 2024, 4:54 AM
tomek accepted D11590: add deletePasswordUser to IdentityServiceClient.
Apr 9 2024, 3:07 AM
tomek accepted D11593: [lib] Introduce `FarcasterRelationshipRequest`, validators, and unit tests.
Apr 9 2024, 3:00 AM
tomek accepted D11592: [lib] Split `RelationshipAction` and `RelationshipActionSansFarcaster`.
Apr 9 2024, 2:56 AM
tomek accepted D11571: Turn migration 38 into a no-op.
Apr 9 2024, 2:53 AM
tomek added inline comments to D11571: Turn migration 38 into a no-op.
Apr 9 2024, 12:52 AM
tomek added inline comments to D11593: [lib] Introduce `FarcasterRelationshipRequest`, validators, and unit tests.
Apr 9 2024, 12:32 AM
tomek added inline comments to D8566: Upload/download large notification payload from the keyserver/Android NSE if its size exceeds FCM limits.
Apr 9 2024, 12:23 AM
tomek added inline comments to D9091: Schedule blob deletion on iOS.
Apr 9 2024, 12:19 AM

Apr 8 2024

tomek added inline comments to D11534: [web][native] Extract the common logic determining when to clear the data.
Apr 8 2024, 8:48 AM
tomek added inline comments to D11533: [web][native] Reset the state iff current user ID changes.
Apr 8 2024, 8:36 AM
tomek added inline comments to D11533: [web][native] Reset the state iff current user ID changes.
Apr 8 2024, 7:05 AM
tomek accepted D9069: Schedule blob deletion on Android when large notifications arrives.
Apr 8 2024, 6:46 AM
tomek accepted D9091: Schedule blob deletion on iOS.
Apr 8 2024, 6:42 AM
tomek requested review of D11573: [native] Clear auth metadata on logout.
Apr 8 2024, 6:17 AM

Apr 3 2024

tomek requested review of D11534: [web][native] Extract the common logic determining when to clear the data.
Apr 3 2024, 5:12 AM
tomek requested review of D11533: [web][native] Reset the state iff current user ID changes.
Apr 3 2024, 4:39 AM
tomek requested review of D11532: [web][native] Refactor reset state logic in reducers.
Apr 3 2024, 3:43 AM
tomek abandoned D4067: [web, lib, native] [refactor] [ENG-536] rename listData to sidebar.

The task from the title https://linear.app/comm/issue/ENG-536/sidebar-list-modal-re-style - is marked as done. If we decide to do something more about that issue, it would be probably easier to do it from scratch instead of figuring out how to make this stack solve the problem.

Apr 3 2024, 2:36 AM
tomek commandeered D4067: [web, lib, native] [refactor] [ENG-536] rename listData to sidebar.
Apr 3 2024, 2:36 AM
tomek abandoned D4066: [web] [refactor] [ENG-536] remove old sidebar styling from side bar modal.

The task from the title https://linear.app/comm/issue/ENG-536/sidebar-list-modal-re-style - is marked as done. If we decide to do something more about that issue, it would be probably easier to do it from scratch instead of figuring out how to make this stack solve the problem.

Apr 3 2024, 2:36 AM
tomek commandeered D4066: [web] [refactor] [ENG-536] remove old sidebar styling from side bar modal.
Apr 3 2024, 2:36 AM
tomek abandoned D4061: [lib, web] [feat] [ENG-536] hook up sidebar data to search modal.

The task from the title https://linear.app/comm/issue/ENG-536/sidebar-list-modal-re-style - is marked as done. If we decide to do something more about that issue, it would be probably easier to do it from scratch instead of figuring out how to make this stack solve the problem.

Apr 3 2024, 2:35 AM
tomek commandeered D4061: [lib, web] [feat] [ENG-536] hook up sidebar data to search modal.
Apr 3 2024, 2:35 AM
tomek abandoned D4060: [native, lib, web] [refactor] move text out of state.

Later in the stack, it is mentioned that this is a part of https://linear.app/comm/issue/ENG-536/sidebar-list-modal-re-style - which is marked as done. If we decide to do something more about that issue, it would be probably easier to do it from scratch instead of figuring out how to make this stack solve the problem.

Apr 3 2024, 2:34 AM
tomek commandeered D4060: [native, lib, web] [refactor] move text out of state.
Apr 3 2024, 2:33 AM
tomek added inline comments to D11530: [scripts] Improve prompt.
Apr 3 2024, 2:10 AM

Apr 2 2024

tomek accepted D11504: [lib] Allow keyserverAuth to update currentAsOf.
Apr 2 2024, 3:11 AM
tomek accepted D11503: [web] Add invalidSessionRecovery checks for web.
Apr 2 2024, 3:10 AM
tomek accepted D11502: [lib] Make invalidSessionRecovery work with all recoveryActionSources handled by KeyserverConnectionHandler.
Apr 2 2024, 3:02 AM
tomek accepted D11501: [keyserver] Don't block innerPerformAuth return on sleep.
Apr 2 2024, 2:47 AM
tomek accepted D11500: [lib] Use identity service for keyserver session recovery.
Apr 2 2024, 2:42 AM
tomek accepted D11499: [lib] Factor out innerPerformAuth for use in keyserver session recovery.
Apr 2 2024, 2:29 AM
tomek accepted D11498: [lib] Expose callKeyserverEndpoint in CallKeyserverEndpointContext.
Apr 2 2024, 2:15 AM
tomek accepted D11497: [lib] Don't invalidate a session if the recovery is cancelled.
Apr 2 2024, 2:10 AM

Mar 29 2024

tomek closed D11440: [lib] Set default state when wiping out a keyserver store.
Mar 29 2024, 6:52 AM
tomek committed rCOMMb174bc063ee3: [lib] Set default state when wiping out a keyserver store (authored by tomek).
[lib] Set default state when wiping out a keyserver store
Mar 29 2024, 6:52 AM

Mar 28 2024

tomek requested review of D11440: [lib] Set default state when wiping out a keyserver store.
Mar 28 2024, 8:34 AM
tomek closed D11369: [lib] Create a wrapper over dispatch.
Mar 28 2024, 8:11 AM
tomek closed D11368: [native][web] Remove processing the ops from reducers.
Mar 28 2024, 8:11 AM
tomek committed rCOMM8a655eac7f4e: [lib] Create a wrapper over dispatch (authored by tomek).
[lib] Create a wrapper over dispatch
Mar 28 2024, 8:11 AM
tomek closed D11367: [lib] Handle processing of the ops.
Mar 28 2024, 8:11 AM
tomek committed rCOMM6c78d160c2b4: [native][web] Remove processing the ops from reducers (authored by tomek).
[native][web] Remove processing the ops from reducers
Mar 28 2024, 8:11 AM
tomek committed rCOMMf6b18846b255: [lib] Handle processing of the ops (authored by tomek).
[lib] Handle processing of the ops
Mar 28 2024, 8:11 AM
tomek closed D11366: [lib] Introduce a handler that processes the ops.
Mar 28 2024, 8:11 AM
tomek committed rCOMM4f549a94bfb2: [lib] Introduce a handler that processes the ops (authored by tomek).
[lib] Introduce a handler that processes the ops
Mar 28 2024, 8:11 AM
tomek closed D11281: [lib] Add an optional ID field to all the actions.
Mar 28 2024, 8:11 AM