Page MenuHomePhabricator
Feed Advanced Search

Jan 29 2024

atul updated the diff for D10791: [lib] Rename `MinimallyEncodedMemberInfo` to `MemberInfo`.

rebase fix import

Jan 29 2024, 2:13 PM
atul updated the diff for D10734: [lib] Rename `MinimallyEncodedRoleInfo` to `RoleInfo`.

rebase: fix import

Jan 29 2024, 2:07 PM
atul updated the diff for D10731: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`.

fix import

Jan 29 2024, 1:06 PM
atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 29 2024, 1:04 PM
atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 29 2024, 1:03 PM
atul updated the diff for D10791: [lib] Rename `MinimallyEncodedMemberInfo` to `MemberInfo`.

rebase

Jan 29 2024, 1:02 PM
atul updated the diff for D10790: [lib] Remove single usage of `LegacyMemberInfo`.

rebase

Jan 29 2024, 1:01 PM
atul updated the diff for D10789: [lib] Expand `MemberInfo` to `LegacyMemberInfo | MinimallyEncodedMemberInfo`.

rebase

Jan 29 2024, 1:01 PM
atul updated the diff for D10734: [lib] Rename `MinimallyEncodedRoleInfo` to `RoleInfo`.

rebase

Jan 29 2024, 1:01 PM
atul updated the diff for D10733: [lib] Remove `LegacyRoleInfo` where possible in `lib`.

rebase

Jan 29 2024, 1:00 PM
atul updated the diff for D10731: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`.

rebase

Jan 29 2024, 1:00 PM
atul closed D10730: Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo`.
Jan 29 2024, 12:44 PM
atul committed rCOMMf36b8ff65079: Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo` (authored by atul).
Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo`
Jan 29 2024, 12:44 PM
atul closed D10729: [lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase.
Jan 29 2024, 12:44 PM
atul committed rCOMM73d2fd7b4e3f: [lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase (authored by atul).
[lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase
Jan 29 2024, 12:44 PM
atul closed D10728: [web][native] Remove client usages of `LegacyRelativeMemberInfo`.
Jan 29 2024, 12:44 PM
atul committed rCOMMcb3ac5eb2528: [web][native] Remove client usages of `LegacyRelativeMemberInfo` (authored by atul).
[web][native] Remove client usages of `LegacyRelativeMemberInfo`
Jan 29 2024, 12:43 PM
atul closed D10727: [lib] First pass at removing `LegacyRelativeMemberInfo` from `lib`.
Jan 29 2024, 12:43 PM
atul committed rCOMMb1012bec65f2: [lib] First pass at removing `LegacyRelativeMemberInfo` from `lib` (authored by atul).
[lib] First pass at removing `LegacyRelativeMemberInfo` from `lib`
Jan 29 2024, 12:43 PM
atul added a comment to D10729: [lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase.

Double checked for import issues before landing.

Jan 29 2024, 12:32 PM
atul added a comment to D10730: Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo`.

Double checked for import issues before landing.

Jan 29 2024, 12:31 PM
atul added a comment to D10728: [web][native] Remove client usages of `LegacyRelativeMemberInfo`.

Double checked for import issues before landing.

Jan 29 2024, 12:30 PM
atul added a comment to D10727: [lib] First pass at removing `LegacyRelativeMemberInfo` from `lib`.

Double checked for import issues before landing.

Jan 29 2024, 12:30 PM
atul updated the diff for D10730: Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo`.

rebase

Jan 29 2024, 12:29 PM
atul updated the diff for D10729: [lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase.

rebase

Jan 29 2024, 12:18 PM
atul updated the diff for D10728: [web][native] Remove client usages of `LegacyRelativeMemberInfo`.

rebase

Jan 29 2024, 12:18 PM
atul updated the diff for D10727: [lib] First pass at removing `LegacyRelativeMemberInfo` from `lib`.

rebase

Jan 29 2024, 12:17 PM
atul closed D10726: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo`.
Jan 29 2024, 11:54 AM
atul committed rCOMM874cc72b1f49: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo |… (authored by atul).
Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo |…
Jan 29 2024, 11:54 AM
atul added a comment to D10726: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo`.

Double checked imports before landing.

Jan 29 2024, 10:03 AM
atul updated the diff for D10726: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo`.

re-introduce fixes

Jan 29 2024, 10:01 AM
atul updated the diff for D10726: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo`.

resolve merge conflicts

Jan 29 2024, 9:34 AM
atul closed D10725: [lib] Remove unused utilities and validators.
Jan 29 2024, 9:32 AM
atul committed rCOMM23b646b949de: [lib] Remove unused utilities and validators (authored by atul).
[lib] Remove unused utilities and validators
Jan 29 2024, 9:32 AM
atul added a comment to D10725: [lib] Remove unused utilities and validators.

Double checked that there were no import issues before landing.

Jan 29 2024, 9:31 AM
atul added a comment to D10703: Prepare for new connection handling.

I updated this diff to disable WAL on web. If you @ashoat or @atul meant to disable WAL across all platforms just let me know and I will change the behaviour.

I'm not sure if @atul's suggestion in this comment was meant to be web-only or across the whole codebase.

Personally, I don't feel strongly. Are there any risks in having a different approach on web vs. native?

Jan 29 2024, 8:00 AM
atul updated the test plan for D10826: [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded.
Jan 29 2024, 7:37 AM
atul added a comment to D10826: [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded.

Unmarking as DRAFT, think this is good for review.

Jan 29 2024, 7:36 AM
atul retitled D10826: [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded from [DRAFT][web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded to [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded.
Jan 29 2024, 7:36 AM

Jan 26 2024

atul updated the diff for D10725: [lib] Remove unused utilities and validators.

rebase

Jan 26 2024, 11:59 PM
atul closed D10724: [lib] Remove simple usages of `LegacyRawThreadInfo`.
Jan 26 2024, 11:48 PM
atul committed rCOMM51c38a95316e: [lib] Remove simple usages of `LegacyRawThreadInfo` (authored by atul).
[lib] Remove simple usages of `LegacyRawThreadInfo`
Jan 26 2024, 11:48 PM
atul updated the diff for D10724: [lib] Remove simple usages of `LegacyRawThreadInfo`.

rebase

Jan 26 2024, 11:27 PM
atul closed D10721: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`.
Jan 26 2024, 11:13 PM
atul committed rCOMMe231ab33123b: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator` (authored by atul).
[lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`
Jan 26 2024, 11:13 PM
atul added a comment to D10721: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`.

Double checked that import ordering issues were resolved before landing.

Jan 26 2024, 11:12 PM
atul updated the diff for D10721: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`.

rebase

Jan 26 2024, 11:11 PM
atul added inline comments to D10721: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`.
Jan 26 2024, 10:58 PM
atul attached a referenced file: F1092352: 44fbca.png.
Jan 26 2024, 10:57 PM
atul attached a referenced file: F1092353: 461d24.png.
Jan 26 2024, 10:57 PM
atul added inline comments to D10721: [lib] Remove `LegacyThreadInfo` and `legacyThreadInfoValidator`.
Jan 26 2024, 10:57 PM
atul added a comment to D10720: [lib] Remove `LegacyThreadInfo`-related type utilities.

Dang rebased off of the wrong branch for this (everything is correct except import ordering). Will double check rest of stack to make sure diffs are updated properly.

Jan 26 2024, 10:47 PM
atul abandoned D10846: [IGNORE] Repro for ENG-6596.
Jan 26 2024, 10:45 PM
atul closed D10720: [lib] Remove `LegacyThreadInfo`-related type utilities.
Jan 26 2024, 4:48 PM
atul committed rCOMM9c89cbd44f3f: [lib] Remove `LegacyThreadInfo`-related type utilities (authored by atul).
[lib] Remove `LegacyThreadInfo`-related type utilities
Jan 26 2024, 4:48 PM
atul updated the diff for D10720: [lib] Remove `LegacyThreadInfo`-related type utilities.

rebase

Jan 26 2024, 4:46 PM
atul closed D10718: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)`.
Jan 26 2024, 3:32 PM
atul committed rCOMM0173927531a1: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)` (authored by atul).
[lib] Remove branching from `threadInfoFromRawThreadInfo(...)`
Jan 26 2024, 3:32 PM
atul updated the diff for D10718: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)`.

land

Jan 26 2024, 2:32 PM
atul requested review of D10846: [IGNORE] Repro for ENG-6596.
Jan 26 2024, 1:09 PM
atul added inline comments to D10718: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)`.
Jan 26 2024, 1:02 PM

Jan 25 2024

atul requested review of D10826: [web] Migrate existing `ClientDBThreadInfo`s on `web` to be minimally encoded.
Jan 25 2024, 2:46 PM
atul accepted D10419: [web] ens resolve usernames in the reaction tooltip.
Jan 25 2024, 1:12 PM
atul added a comment to D10820: [web] introduce previouslySelectedUsers map to add users list.

What does "vip" mean in this situation? Not seeing anything in the linked Linear issue.

Jan 25 2024, 1:10 PM
atul updated the test plan for D10812: Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`.
Jan 25 2024, 11:39 AM
atul requested review of D10812: Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`.

Updated Test Plan

Jan 25 2024, 11:38 AM
atul requested review of D10818: [lib] Add `threadInfoFromRawThreadInfo` test to `thread-utils.test`.
Jan 25 2024, 11:37 AM
atul attached a referenced file: F1088546: 6b5333.png.
Jan 25 2024, 11:29 AM
atul attached a referenced file: F1088547: d37ed4.png.
Jan 25 2024, 11:29 AM
atul requested review of D10718: [lib] Remove branching from `threadInfoFromRawThreadInfo(...)`.
Jan 25 2024, 11:28 AM
atul added a comment to D10810: Remove branching on `.minimallyEncoded` where no longer relevant.

Can you remove the extra newlines before landing? I'm assuming they're not intentional

Jan 25 2024, 8:35 AM

Jan 24 2024

atul requested review of D10812: Rename `useDisconnectedBarVisibilityHandler` to `useKeyserverReachabilityHandler`.
Jan 24 2024, 11:06 PM
atul requested review of D10811: [web][native] Remove branching on `minimallyEncoded` in `redux-setup`.
Jan 24 2024, 10:43 PM
atul requested review of D10810: Remove branching on `.minimallyEncoded` where no longer relevant.
Jan 24 2024, 10:38 PM
atul requested review of D10807: [lib] Remove support for `LegacyRawThreadInfo` in `getRelativeMemberInfos`.
Jan 24 2024, 10:14 PM
atul requested review of D10806: Get rid of `minimallyEncoded*` prefix for validators.
Jan 24 2024, 9:57 PM
atul requested review of D10805: Rename `MinimallyEncodedThreadCurrentUserInfo` to `ThreadCurrentUserInfo`.
Jan 24 2024, 9:41 PM
atul updated the diff for D10791: [lib] Rename `MinimallyEncodedMemberInfo` to `MemberInfo`.

rebase

Jan 24 2024, 9:17 PM
atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 24 2024, 8:22 PM
atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 24 2024, 8:21 PM
atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 24 2024, 8:12 PM
atul updated the diff for D10791: [lib] Rename `MinimallyEncodedMemberInfo` to `MemberInfo`.

rebase

Jan 24 2024, 8:11 PM
atul updated the diff for D10790: [lib] Remove single usage of `LegacyMemberInfo`.

rebase

Jan 24 2024, 8:11 PM
atul updated the diff for D10789: [lib] Expand `MemberInfo` to `LegacyMemberInfo | MinimallyEncodedMemberInfo`.

rebase

Jan 24 2024, 8:11 PM
atul accepted D10746: [web] introduce clear all button to add users list.

Seems reasonable, thanks for pointing out other places in the codebase that use that pattern.

Jan 24 2024, 7:53 PM

Jan 23 2024

atul updated the diff for D10792: Rename `ThreadCurrentUserInfo` to `LegacyThreadCurrentUserInfo`.

rebase

Jan 23 2024, 2:39 PM
atul updated the diff for D10791: [lib] Rename `MinimallyEncodedMemberInfo` to `MemberInfo`.

rebase

Jan 23 2024, 2:39 PM
atul updated the diff for D10790: [lib] Remove single usage of `LegacyMemberInfo`.

rebase

Jan 23 2024, 2:39 PM
atul updated the diff for D10789: [lib] Expand `MemberInfo` to `LegacyMemberInfo | MinimallyEncodedMemberInfo`.

rebase

Jan 23 2024, 2:38 PM
atul updated the diff for D10789: [lib] Expand `MemberInfo` to `LegacyMemberInfo | MinimallyEncodedMemberInfo`.

rebase

Jan 23 2024, 2:38 PM
atul updated the diff for D10734: [lib] Rename `MinimallyEncodedRoleInfo` to `RoleInfo`.

rebase

Jan 23 2024, 2:37 PM
atul updated the diff for D10733: [lib] Remove `LegacyRoleInfo` where possible in `lib`.

rebase

Jan 23 2024, 2:37 PM
atul updated the diff for D10731: [lib] Expand `RoleInfo` to `LegacyRoleInfo | MinimallyEncodedRoleInfo`.

rebase

Jan 23 2024, 2:37 PM
atul updated the diff for D10730: Rename `MinimallyEncodedRelativeMemberInfo` to `RelativeMemberInfo`.

rebase

Jan 23 2024, 2:36 PM
atul updated the diff for D10729: [lib] Remove all remaining instances of `LegacyRelativeMemberInfo` from codebase.

rebase

Jan 23 2024, 2:36 PM
atul updated the diff for D10728: [web][native] Remove client usages of `LegacyRelativeMemberInfo`.

rebase

Jan 23 2024, 2:36 PM
atul updated the diff for D10727: [lib] First pass at removing `LegacyRelativeMemberInfo` from `lib`.

rebase

Jan 23 2024, 2:35 PM
atul updated the diff for D10726: Expand `RelativeMemberInfo` to `LegacyRelativeMemberInfo | MinimallyEncodedRelativeMemberInfo`.

rebase

Jan 23 2024, 2:34 PM
atul updated the diff for D10725: [lib] Remove unused utilities and validators.

rebaser

Jan 23 2024, 2:32 PM
atul updated the diff for D10724: [lib] Remove simple usages of `LegacyRawThreadInfo`.

rebase

Jan 23 2024, 2:32 PM