Page MenuHomePhabricator

ashoat (Ashoat)
UserAdministrator

Projects

User does not belong to any projects.

User Details

User Since
Jul 20 2020, 9:28 AM (226 w, 2 d)
Roles
Administrator

Recent Activity

Today

ashoat updated the diff for D13984: [lib] Fix incorrect timestamps for sidebars.

Changes in initial version broke useSearchSidebars comparison algo. Moved the sorting to getSidebarItems instead, which isn't used in useSearchSidebars and is only used in chat-selectors.js

Wed, Nov 20, 7:15 PM
ashoat published D13984: [lib] Fix incorrect timestamps for sidebars for review.
Wed, Nov 20, 9:08 AM
ashoat updated the diff for D13983: [lib] Don't query Neynar for Farcaster channel avatars if unused.

Fix rebase mistake

Wed, Nov 20, 8:05 AM
ashoat published D13983: [lib] Don't query Neynar for Farcaster channel avatars if unused for review.
Wed, Nov 20, 7:16 AM
ashoat published D13982: [lib][native][web] Introduce FarcasterChannelPrefetchHandler for review.
Wed, Nov 20, 7:15 AM
ashoat published D13981: [lib] Query for channel info in bulk whenever possible for review.
Wed, Nov 20, 7:15 AM
ashoat published D13980: [lib] Introduce NeynarClient.fetchFarcasterChannelsByIDs for review.
Wed, Nov 20, 7:13 AM
ashoat published D13979: [lib] Use channel instead of channel/search Neynar endpoint for review.
Wed, Nov 20, 7:11 AM
ashoat published D13978: [lib] Fix bug in getCachedFarcasterChannelEntryForChannelID for review.
Wed, Nov 20, 7:11 AM
ashoat published D13977: [lib] Wait to finish all followed channels queries before querying for a specific one for review.
Wed, Nov 20, 7:10 AM
ashoat requested review of D13976: [lib][keyserver][native][web] Introduce FCCache.getFollowedFarcasterChannelsForFID.
Wed, Nov 20, 7:04 AM
ashoat requested review of D13975: [keyserver][native][web] Use FCCache instead of calling fetchFarcasterChannelByID directly.
Wed, Nov 20, 6:57 AM
ashoat requested review of D13974: [lib] Get rid of fetchFollowedFarcasterChannelsWithFilter.
Wed, Nov 20, 6:56 AM
ashoat added a comment to D13939: [lib] Consider thread infos in useUsersSupportThickThreads().

Yes, exactly!

Wed, Nov 20, 4:24 AM
ashoat accepted D13963: [lib] create hook to run Device List Update Protocol.
Wed, Nov 20, 4:23 AM
ashoat accepted D13972: [lib] standarize broadcasting device list update when removing device.
Wed, Nov 20, 4:23 AM
ashoat added inline comments to D13972: [lib] standarize broadcasting device list update when removing device.
Wed, Nov 20, 4:23 AM
ashoat accepted D13971: [keyserver] implement invite link button to frog hono route.
Wed, Nov 20, 4:20 AM
ashoat added inline comments to D13947: [keyserver] frog hello world example.
Wed, Nov 20, 4:17 AM
ashoat accepted D13854: [keyserver] return threadInfo in fetchAllCommunityInfosWithNames.
Wed, Nov 20, 4:15 AM
ashoat accepted D13852: [keyserver] new fetchPrivilegedThreadInfos function.
Wed, Nov 20, 4:14 AM

Yesterday

ashoat committed rCOMM64e3d1dad642: [terraform] Bump keyserver image to 1.0.144 (authored by ashoat).
[terraform] Bump keyserver image to 1.0.144
Tue, Nov 19, 12:28 PM
ashoat added a reverting change for rCOMMcd4943fc4074: [native] Temporary changes for staff release: rCOMM579ec909f09a: Revert "[native] Temporary changes for staff release".
Tue, Nov 19, 8:40 AM
ashoat committed rCOMM579ec909f09a: Revert "[native] Temporary changes for staff release" (authored by ashoat).
Revert "[native] Temporary changes for staff release"
Tue, Nov 19, 8:40 AM
ashoat committed rCOMMe371bd8ec52c: [native] codeVersion -> 444 (authored by ashoat).
[native] codeVersion -> 444
Tue, Nov 19, 8:39 AM
ashoat committed rCOMMcd4943fc4074: [native] Temporary changes for staff release (authored by ashoat).
[native] Temporary changes for staff release
Tue, Nov 19, 8:39 AM
ashoat committed rCOMM3c0553e3f578: [native] codeVersion -> 443 (authored by ashoat).
[native] codeVersion -> 443
Tue, Nov 19, 8:39 AM
ashoat committed rCOMM77e5e2667861: [web][keyserver] codeVersion -> 144 (authored by ashoat).
[web][keyserver] codeVersion -> 144
Tue, Nov 19, 8:39 AM
ashoat added inline comments to D13963: [lib] create hook to run Device List Update Protocol.
Tue, Nov 19, 8:30 AM
ashoat added inline comments to D13963: [lib] create hook to run Device List Update Protocol.
Tue, Nov 19, 8:28 AM
ashoat added inline comments to D13963: [lib] create hook to run Device List Update Protocol.
Tue, Nov 19, 5:44 AM
ashoat requested changes to D13946: [native] Update the QR login screen to mention the primary device.

Passing to @tomek's queue

Tue, Nov 19, 3:58 AM

Mon, Nov 18

ashoat accepted D13947: [keyserver] frog hello world example.
Mon, Nov 18, 6:22 PM
ashoat added a comment to D13951: [keyserver] Log on keyserver when INITIAL_NOTIFICATIONS_ENCRYPTED_MESSAGE is sent to the client.

Just wondering, what do you think of this style?

Mon, Nov 18, 6:13 PM
ashoat added a comment to D13939: [lib] Consider thread infos in useUsersSupportThickThreads().

I guess a way to reframe what you're saying about useUsersSupportThickThreads is that userHasDeviceList doesn't match findUserIdentities in the case of an empty device list. We end up returning the correct answer anyways, since we check findUserIdentities as a fallback. But it would be better if we were able to get the right answer without having to query the identity service.

Mon, Nov 18, 6:01 PM
ashoat requested changes to D13940: [lib] Don't swallow errors in getUserIdentities() call.

Thanks @angelika. Some concerns:

Mon, Nov 18, 5:42 AM

Fri, Nov 15

ashoat accepted D13949: [keyserver][lib] change thread info validators -> thin raw thread info validators.

I went through each of these on my own and I think it's safe

Fri, Nov 15, 2:05 PM
ashoat requested changes to D13947: [keyserver] frog hello world example.

Back to you with question

Fri, Nov 15, 2:03 PM
ashoat added inline comments to D13947: [keyserver] frog hello world example.
Fri, Nov 15, 2:02 PM
ashoat accepted D13906: [native] Introduce a restore button.

On the web, we don't support the restoration flow, so I don't think it makes sense to introduce the link to the restore flow.

Fri, Nov 15, 10:40 AM
ashoat requested changes to D13852: [keyserver] new fetchPrivilegedThreadInfos function.

Don't love that we're artifically inserting a permission that doesn't exist. Is it truly necessary?

Fri, Nov 15, 10:31 AM
ashoat closed D13925: [landing] Add Rahul to landing page.
Fri, Nov 15, 10:25 AM
ashoat committed rCOMM7960f46f08e0: [landing] Add Rahul to landing page (authored by ashoat).
[landing] Add Rahul to landing page
Fri, Nov 15, 10:25 AM
ashoat added a comment to D13906: [native] Introduce a restore button.

Looks great!

Fri, Nov 15, 10:23 AM

Thu, Nov 14

ashoat added inline comments to D13919: [lib] Don't bump thin thread timestamp for membership robotext.
Thu, Nov 14, 4:53 AM
ashoat retitled D13919: [lib] Don't bump thin thread timestamp for membership robotext from [lib] Don't bump thick thread timestamp for membership robotext to [lib] Don't bump thin thread timestamp for membership robotext.
Thu, Nov 14, 4:53 AM
ashoat added a comment to D13906: [native] Introduce a restore button.

To be honest, I think it looks okay without the underline. What do you think?

Thu, Nov 14, 4:52 AM

Wed, Nov 13

ashoat accepted D13926: [keyserver] add frog, hono, @hono/node-server as keyserver dependencies.
Wed, Nov 13, 12:50 PM
ashoat accepted D13927: [keyserver] Have @babel/preset-react use automatic runtime.
Wed, Nov 13, 12:50 PM
ashoat requested review of D13925: [landing] Add Rahul to landing page.
Wed, Nov 13, 12:13 PM
ashoat closed D13916: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different.
Wed, Nov 13, 11:20 AM
ashoat closed D13915: [lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime.
Wed, Nov 13, 11:20 AM
ashoat committed rCOMMed9e9cf2c268: [lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime (authored by ashoat).
[lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM4d3afd3231b5: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different (authored by ashoat).
[lib][web] Await lastUpdatedTime and update ChatThreadItems if different
Wed, Nov 13, 11:20 AM
ashoat closed D13914: [lib] Extract getSidebarItems.
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM481690308381: [lib] Extract getSidebarItems (authored by ashoat).
[lib] Extract getSidebarItems
Wed, Nov 13, 11:20 AM
ashoat closed D13913: [lib] Introduce MessageSpec.getLastUpdatedTime.
Wed, Nov 13, 11:20 AM
ashoat closed D13912: [lib] Extract getLastUpdatedTime to a separate file.
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM642c7eb1e0a3: [lib] Introduce MessageSpec.getLastUpdatedTime (authored by ashoat).
[lib] Introduce MessageSpec.getLastUpdatedTime
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM084e0062ebd5: [lib] Extract getLastUpdatedTime to a separate file (authored by ashoat).
[lib] Extract getLastUpdatedTime to a separate file
Wed, Nov 13, 11:20 AM
ashoat closed D13911: [lib] Convert sidebarInfoSelector to a hook.
Wed, Nov 13, 11:20 AM
ashoat closed D13921: [lib] Move sidebarInfoSelector to different folder.
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM0612ebfa0ff2: [lib] Convert sidebarInfoSelector to a hook (authored by ashoat).
[lib] Convert sidebarInfoSelector to a hook
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM1d7782667623: [lib] Move sidebarInfoSelector to different folder (authored by ashoat).
[lib] Move sidebarInfoSelector to different folder
Wed, Nov 13, 11:20 AM
ashoat closed D13910: [lib] Inline getMostRecentMessageInfo.
Wed, Nov 13, 11:20 AM
ashoat committed rCOMM4cfad4a5f144: [lib] Inline getMostRecentMessageInfo (authored by ashoat).
[lib] Inline getMostRecentMessageInfo
Wed, Nov 13, 11:20 AM
ashoat added inline comments to D13919: [lib] Don't bump thin thread timestamp for membership robotext.
Wed, Nov 13, 11:15 AM
ashoat added inline comments to D13919: [lib] Don't bump thin thread timestamp for membership robotext.
Wed, Nov 13, 11:13 AM
ashoat updated the diff for D13916: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different.

Use Array.prototype.every()

Wed, Nov 13, 11:01 AM
ashoat added inline comments to D13916: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different.
Wed, Nov 13, 11:01 AM
ashoat updated the diff for D13913: [lib] Introduce MessageSpec.getLastUpdatedTime.

Code comments

Wed, Nov 13, 10:56 AM
ashoat added inline comments to D13913: [lib] Introduce MessageSpec.getLastUpdatedTime.
Wed, Nov 13, 10:55 AM
ashoat added inline comments to D13915: [lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime.
Wed, Nov 13, 10:34 AM
ashoat added a comment to D13906: [native] Introduce a restore button.

I was more concerned about users being able to notice that a link can be clickable. In the mobile app, we usually use buttons to indicate actions that can be performed. There are some usages of LinkButton, but only in the navigation header where users expect some actions to be possible.

Wed, Nov 13, 7:49 AM
ashoat added a comment to D13906: [native] Introduce a restore button.

It sounds like you're proposing a solution where on the initial screen there are two buttons: Sign in and Register. Clicking Sign in opens the QR screen, where there is a No longer have access to your primary device? text. Clicking this link would open the restore screen. Is that correct?

Wed, Nov 13, 7:25 AM
ashoat accepted D13922: [sqlite] Add missing spaces in getInboundP2PMessagesByID query.
Wed, Nov 13, 7:23 AM
ashoat accepted D13923: [lib] Fix capitalization in usePeerToPeerMessageHandler.
Wed, Nov 13, 7:23 AM
ashoat accepted D13924: [lib] Fix long line in olmSessionErrors.
Wed, Nov 13, 7:23 AM
ashoat added inline comments to D13901: [lib] Do not reset a session when getting already decrypted message.
Wed, Nov 13, 5:59 AM
ashoat added inline comments to D13897: [sqlite] Add getInboundP2PMessagesByID query.
Wed, Nov 13, 5:49 AM

Tue, Nov 12

ashoat requested review of D13921: [lib] Move sidebarInfoSelector to different folder.
Tue, Nov 12, 3:40 PM
ashoat requested review of D13919: [lib] Don't bump thin thread timestamp for membership robotext.
Tue, Nov 12, 3:33 PM
ashoat requested review of D13918: [lib] Reactions only bump thread timestamps when reacting to viewer.
Tue, Nov 12, 3:24 PM
ashoat requested review of D13916: [lib][web] Await lastUpdatedTime and update ChatThreadItems if different.
Tue, Nov 12, 3:16 PM
ashoat requested review of D13915: [lib][native][web] Update useSidebarInfos to include lastUpdatedAtLeastTime.
Tue, Nov 12, 3:08 PM
ashoat updated the summary of D13911: [lib] Convert sidebarInfoSelector to a hook.
Tue, Nov 12, 2:51 PM
ashoat updated the summary of D13911: [lib] Convert sidebarInfoSelector to a hook.
Tue, Nov 12, 2:51 PM
ashoat updated the summary of D13911: [lib] Convert sidebarInfoSelector to a hook.
Tue, Nov 12, 2:50 PM
ashoat updated the diff for D13911: [lib] Convert sidebarInfoSelector to a hook.

Rebase

Tue, Nov 12, 2:49 PM
ashoat updated the summary of D13911: [lib] Convert sidebarInfoSelector to a hook.
Tue, Nov 12, 2:48 PM
ashoat requested review of D13914: [lib] Extract getSidebarItems.
Tue, Nov 12, 2:18 PM
ashoat requested review of D13913: [lib] Introduce MessageSpec.getLastUpdatedTime.
Tue, Nov 12, 2:10 PM
ashoat accepted D13917: [native] make the list of NUX tips to be displayed configurable.

Looks good to me! I'm not familiar with the NUX code (it was introduced while I was away) so would appreciate if @tomek could take a look too

Tue, Nov 12, 2:01 PM
ashoat requested review of D13912: [lib] Extract getLastUpdatedTime to a separate file.
Tue, Nov 12, 1:07 PM
ashoat requested review of D13911: [lib] Convert sidebarInfoSelector to a hook.
Tue, Nov 12, 1:06 PM
ashoat requested review of D13910: [lib] Inline getMostRecentMessageInfo.
Tue, Nov 12, 1:02 PM
ashoat added a comment to D13907: [lib] Introduce a restore screen.
  1. I'm not sure the "OR" thing is necessary anymore now that we only have two buttons instead of three
  2. Can we replace "Sign in" with "Sign in with password"?
Tue, Nov 12, 7:25 AM
ashoat added a comment to D13906: [native] Introduce a restore button.

I think it would be best to tuck the restore button behind the login button.

Tue, Nov 12, 7:24 AM
ashoat added inline comments to D13901: [lib] Do not reset a session when getting already decrypted message.
Tue, Nov 12, 6:50 AM
ashoat closed D13903: [lib] Avoid creation new ChatThreadItem unnecessarily in getSearchResultsForEmptySearchText.
Tue, Nov 12, 6:49 AM