Page MenuHomePhorge
Feed Search

Sep 21 2023

atul added inline comments to D9262: [lib] Add `ThreadActivityStore` to Redux.
Sep 21 2023, 9:37 PM
atul updated the diff for D9262: [lib] Add `ThreadActivityStore` to Redux.

fix

Sep 21 2023, 9:33 PM
atul updated the diff for D9262: [lib] Add `ThreadActivityStore` to Redux.

address feedback

Sep 21 2023, 9:30 PM
atul added a comment to D9262: [lib] Add `ThreadActivityStore` to Redux.

In the past, we would need to update websites-responders.js when adding a new field to Redux state. Now that Redux state for the web is fetched via an API call, is there a different place we should be updating?

Sep 21 2023, 9:22 PM
atul published D9262: [lib] Add `ThreadActivityStore` to Redux for review.
Sep 21 2023, 8:58 PM
atul updated the diff for D9227: [patch] Patch `expo-modules-core` to unblock iOS build on macOS Sonoma.

rebase

Sep 21 2023, 5:55 PM

Sep 20 2023

atul committed rCOMM1cf8c3d6ba22: Revert "[native] Temporary changes for staff release".
Sep 20 2023, 9:35 PM
atul committed rCOMM9d0a6d545307: [native] `codeVersion` -> 264.
Sep 20 2023, 9:35 PM
atul committed rCOMMe8bd73f1d8e1: [native] Temporary changes for staff release.
Sep 20 2023, 9:35 PM
atul committed rCOMM87443cc76daf: [native] `codeVersion` -> 263.
Sep 20 2023, 9:35 PM
atul added reviewers for D9235: [lib] introduce createPendingPersonalThread function: ashoat, tomek.

Adding @ashoat and @tomek here since I don't have a ton of context on pending threads

Sep 20 2023, 8:54 PM
atul added a comment to D9251: [keyserver] Remove unused exports in message-fetchers.js.

Android failure is unrelated

Sep 20 2023, 8:54 PM
atul accepted D9223: [nix] add logs to console while waiting for MariaDB to come up.

Looks good, left comments but they're purely personal preference so feel free to land as is

Sep 20 2023, 8:52 PM
atul accepted D9120: [native] update UserProfileBottomSheet to get a userInfo from a userID.
Sep 20 2023, 8:51 PM
atul accepted D9203: [web] introduce AppearanceChangeModal.
Sep 20 2023, 8:50 PM
atul added inline comments to D9248: [native] Fix rescinds and badge-only notifs on iOS.
Sep 20 2023, 5:45 PM
atul added a comment to D9227: [patch] Patch `expo-modules-core` to unblock iOS build on macOS Sonoma.

Can you create a GitHub issue on the Expo repo and link it here before landing?

Sep 20 2023, 5:39 PM
atul accepted D9248: [native] Fix rescinds and badge-only notifs on iOS.

Was there maybe some corresponding keyserver change that needed to be deployed before this release or something?

Sep 20 2023, 5:37 PM
atul published D9248: [native] Fix rescinds and badge-only notifs on iOS for review.
Sep 20 2023, 5:36 PM
atul closed D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.
Sep 20 2023, 5:31 PM
atul committed rCOMM125b29a6d73e: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.
Sep 20 2023, 5:31 PM
atul updated the diff for D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.

toggle withEnableSampleProfiling on DEBUG

Sep 20 2023, 5:24 PM
atul added a comment to D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.

Okay, please make sure that upstream React Native does this on all builds too before landing!

Sep 20 2023, 5:21 PM
atul added a comment to D9227: [patch] Patch `expo-modules-core` to unblock iOS build on macOS Sonoma.

Is there a GitHub issue in Expo's GitHub repo for this?

Sep 20 2023, 4:47 PM

Sep 19 2023

atul committed rCOMMebb529b9ae7b: Revert "[native] Temporary changes for staff release".
Sep 19 2023, 10:05 PM
atul committed rCOMM1393b9e9c983: [native] `codeVersion` -> 262.
Sep 19 2023, 10:05 PM
atul committed rCOMMbc628b9e3b8d: [native] Temporary changes for staff release.
Sep 19 2023, 10:05 PM
atul committed rCOMM208984aaf533: [native] `codeVersion` -> 261.
Sep 19 2023, 10:05 PM
atul committed rCOMMbb73b304a2e6: Bump webpki from 0.22.0 to 0.22.1 in /services/feature-flags (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:58 PM
atul committed rCOMMeb3e37ff76d6: Bump webpki from 0.22.0 to 0.22.1 in /services/backup (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:58 PM
atul committed rCOMM5b994dbc623d: Bump webpki from 0.22.0 to 0.22.1 in /services/comm-services-lib (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:58 PM
atul committed rCOMM88f1887d900f: Bump webpki from 0.22.0 to 0.22.1 in /services/blob (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:58 PM
atul committed rCOMM92f52ec682b5: Bump webpki from 0.22.0 to 0.22.1 in /services/tunnelbroker (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:57 PM
atul committed rCOMM50c9eb9535d9: Bump webpki from 0.22.0 to 0.22.1 in /services/reports (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:57 PM
atul committed rCOMMe52697f28480: Bump webpki from 0.22.0 to 0.22.1 in /services/commtest (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:56 PM
atul committed rCOMMd2f9770dca2b: Bump electron from 25.3.0 to 25.8.1 (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:56 PM
atul committed rCOMM55f60fee8c1a: Bump webpki from 0.22.0 to 0.22.1 in /services/identity (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:56 PM
atul committed rCOMMb2deff08ea76: Bump apollo-server-core from 3.11.1 to 3.12.1 (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:55 PM
atul committed rCOMMcbe88c3eecd2: Bump rustls-webpki from 0.101.3 to 0.101.4 in /services/identity (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:55 PM
atul committed rCOMM2d17a1619894: Bump activesupport from 6.1.7.3 to 6.1.7.6 in /native (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:54 PM
atul committed rCOMM3796da87658b: Bump rustls-webpki in /keyserver/addons/rust-node-addon (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:54 PM
atul committed rCOMMe3acc63103fd: Bump rustls-webpki from 0.101.3 to 0.101.4 in /shared/grpc_clients (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:54 PM
atul committed rCOMMacd5e89c8ec8: Bump rustls-webpki from 0.100.1 to 0.100.2 in /services/tunnelbroker (authored by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>).
Sep 19 2023, 9:53 PM
atul closed D9228: [iOS] Set `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` flag to unblock iOS build.
Sep 19 2023, 9:43 PM
atul committed rCOMMf30690b547dd: [iOS] Set `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` flag to unblock….
Sep 19 2023, 9:43 PM
atul updated the diff for D9228: [iOS] Set `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` flag to unblock iOS build.

rebase and land

Sep 19 2023, 9:35 PM
atul added a comment to D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.

(I think the profiler gets disabled on prod builds, I'll do a little more research)

Sep 19 2023, 8:48 PM
atul added a comment to D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig`.

It seems sketchy to do this in production, especially given it's undocumented. I'm worried about the potential impact on performance. Can we restrict this change to dev builds?

Sep 19 2023, 8:47 PM
atul accepted D9232: [keyserver] Prevent creation of sidebars from pinned message robotext.
Sep 19 2023, 8:40 PM
atul accepted D9231: [keyserver] Move sidebar-of-sidebar check earlier in createThread.
Sep 19 2023, 8:39 PM
atul accepted D9230: [keyserver] Delete sidebars created from pinned message robotexts.
Sep 19 2023, 8:38 PM
atul accepted D9229: [keyserver] Add ignorePermissions option to deleteThread.
Sep 19 2023, 8:35 PM
atul published D9233: [iOS] Add `withEnableSampleProfiling(true)` to `makeRuntimeConfig` for review.
Sep 19 2023, 8:31 PM
atul published D9228: [iOS] Set `_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION` flag to unblock iOS build for review.
Sep 19 2023, 5:15 PM
atul published D9227: [patch] Patch `expo-modules-core` to unblock iOS build on macOS Sonoma for review.
Sep 19 2023, 5:00 PM
atul abandoned D9211: [native] Memoize `flatList` in `ChatThreadList`.

@tomek Sorry, you’re totally right that this is useless. I tested performance with/without this diff in isolation and there was no difference.

Sep 19 2023, 2:33 PM

Sep 18 2023

atul closed D9215: [native] Remove `boundChatListData` from `useThreadListSearch` dep list.
Sep 18 2023, 9:04 PM
atul committed rCOMM84b611f0316c: [native] Remove `boundChatListData` from `useThreadListSearch` dep list.
Sep 18 2023, 9:04 PM
atul closed D9212: [native] Consume `useThreadListSearch` in `ChatThreadList`.
Sep 18 2023, 9:04 PM
atul committed rCOMM74c8424f88cc: [native] Consume `useThreadListSearch` in `ChatThreadList`.
Sep 18 2023, 9:04 PM
atul updated the diff for D9215: [native] Remove `boundChatListData` from `useThreadListSearch` dep list.

rebase around D9211 to land

Sep 18 2023, 8:49 PM
atul updated the diff for D9212: [native] Consume `useThreadListSearch` in `ChatThreadList`.

rebase around D9211 to land

Sep 18 2023, 8:48 PM
atul closed D9210: [native] Add some basic memoization to `ChatThreadListSearch`.
Sep 18 2023, 8:46 PM
atul committed rCOMM90f88e7b7c08: [native] Add some basic memoization to `ChatThreadListSearch`.
Sep 18 2023, 8:46 PM
atul updated the diff for D9210: [native] Add some basic memoization to `ChatThreadListSearch`.

land

Sep 18 2023, 8:45 PM
atul closed D9209: [native] Forward `searchInputRef` from `ChatThreadList` to `ChatThreadListSearch`.
Sep 18 2023, 6:01 PM
atul committed rCOMMb2d1a3f8e4f9: [native] Forward `searchInputRef` from `ChatThreadList` to….
Sep 18 2023, 6:01 PM
atul closed D9208: [native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly.
Sep 18 2023, 6:01 PM
atul closed D9207: [native] Move Reanimated code from `ChatThreadList` to `ChatThreadListSearch`.
Sep 18 2023, 6:01 PM
atul committed rCOMM07a97b02cad6: [native] Move Reanimated code from `ChatThreadList` to `ChatThreadListSearch`.
Sep 18 2023, 6:00 PM
atul committed rCOMMa0967233d046: [native] Remove `renderSearch` in `ChatThreadList` and inline….
Sep 18 2023, 6:00 PM
atul closed D9206: [native] Move rest of Search views to `ChatThreadListSearch`.
Sep 18 2023, 6:00 PM
atul committed rCOMMe9e030d6493b: [native] Move rest of Search views to `ChatThreadListSearch`.
Sep 18 2023, 6:00 PM
atul updated the diff for D9209: [native] Forward `searchInputRef` from `ChatThreadList` to `ChatThreadListSearch`.

merge conflicts

Sep 18 2023, 5:33 PM
atul updated the diff for D9215: [native] Remove `boundChatListData` from `useThreadListSearch` dep list.

rebase

Sep 18 2023, 5:33 PM
atul updated the diff for D9212: [native] Consume `useThreadListSearch` in `ChatThreadList`.

rebase

Sep 18 2023, 5:32 PM
atul updated the diff for D9211: [native] Memoize `flatList` in `ChatThreadList`.

rebase

Sep 18 2023, 5:31 PM
atul updated the diff for D9210: [native] Add some basic memoization to `ChatThreadListSearch`.

rebase

Sep 18 2023, 5:31 PM
atul updated the diff for D9208: [native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly.

rebase

Sep 18 2023, 5:28 PM
atul added inline comments to D9209: [native] Forward `searchInputRef` from `ChatThreadList` to `ChatThreadListSearch`.
Sep 18 2023, 5:27 PM
atul updated the diff for D9208: [native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly.

address feedback: early-return

Sep 18 2023, 5:25 PM
atul added inline comments to D9208: [native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly.
Sep 18 2023, 5:25 PM
atul updated the diff for D9208: [native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly.

rebase

Sep 18 2023, 5:22 PM
atul updated the diff for D9207: [native] Move Reanimated code from `ChatThreadList` to `ChatThreadListSearch`.

rebase

Sep 18 2023, 5:22 PM
atul added inline comments to D9207: [native] Move Reanimated code from `ChatThreadList` to `ChatThreadListSearch`.
Sep 18 2023, 5:20 PM
atul updated the diff for D9206: [native] Move rest of Search views to `ChatThreadListSearch`.

rebase after resolve merge conflict

Sep 18 2023, 5:20 PM
atul added inline comments to D9206: [native] Move rest of Search views to `ChatThreadListSearch`.
Sep 18 2023, 4:16 PM
atul requested review of D9211: [native] Memoize `flatList` in `ChatThreadList`.
Sep 18 2023, 4:15 PM
atul added a comment to D9211: [native] Memoize `flatList` in `ChatThreadList`.
In D9211#271261, @tomek wrote:

Have you tested if memoizing the list is beneficial? FlatList is already memoized internally (PureComponent) so wrapping it with memo shouldn't improve anything, but maybe I'm missing something.

Sep 18 2023, 4:15 PM

Sep 17 2023

atul added inline comments to D9206: [native] Move rest of Search views to `ChatThreadListSearch`.
Sep 17 2023, 11:19 PM
atul closed D9205: [native] Introduce naive `ChatThreadListSearch` component.
Sep 17 2023, 11:15 PM
atul committed rCOMMaa58fb3d076f: [native] Introduce naive `ChatThreadListSearch` component.
Sep 17 2023, 11:15 PM
atul updated the diff for D9205: [native] Introduce naive `ChatThreadListSearch` component.

type fn return values as mixed

Sep 17 2023, 11:13 PM
atul added inline comments to D9205: [native] Introduce naive `ChatThreadListSearch` component.
Sep 17 2023, 11:09 PM
atul closed D9214: [native] Memoize `ConnectedRemoteImage`.
Sep 17 2023, 10:31 PM
atul committed rCOMM5e8672d069eb: [native] Memoize `ConnectedRemoteImage`.
Sep 17 2023, 10:31 PM
atul closed D9213: [native] Memoize `ThreadAncestorsLabel`.
Sep 17 2023, 10:31 PM
atul committed rCOMM3df742db5cd1: [native] Memoize `ThreadAncestorsLabel`.
Sep 17 2023, 10:31 PM
atul updated the diff for D9214: [native] Memoize `ConnectedRemoteImage`.

rebase and land

Sep 17 2023, 10:28 PM
atul closed D9219: [native] Bump `react-native` to `0.70.9`.
Sep 17 2023, 10:28 PM