Page MenuHomePhabricator
Feed Advanced Search

Mar 24 2023

ashoat requested changes to D7166: [lib][keyserver] Attach MIME and encryption key to uploads.

Passing back to you with question about multiple files being uploaded to the same endpoint at once. I don't think we actually ever do this, so I'm wondering if we can simply throw an exception if we get multiple files in the same upload

Mar 24 2023, 11:41 AM
ashoat added inline comments to D7166: [lib][keyserver] Attach MIME and encryption key to uploads.
Mar 24 2023, 11:00 AM
ashoat requested changes to D7129: [lib] Added new 'sendEditMessage' action.
Mar 24 2023, 10:57 AM
ashoat requested changes to D7143: [lib] Function to dispatch edit message action.
Mar 24 2023, 10:54 AM
ashoat added inline comments to D7143: [lib] Function to dispatch edit message action.
Mar 24 2023, 10:54 AM
ashoat added inline comments to D7140: [lib] Displaying edited content in a pending thread.
Mar 24 2023, 10:52 AM
ashoat attached a referenced file: F443322: Screenshot 2023-03-24 at 10.28.41 AM.png.
Mar 24 2023, 9:22 AM
ashoat attached a referenced file: F443323: Screenshot 2023-03-24 at 12.18.25 PM.png.
Mar 24 2023, 9:22 AM
ashoat added inline comments to D7076: [keyserver] Add search table to the db.
Mar 24 2023, 9:22 AM

Mar 23 2023

ashoat requested review of D7165: [lib] Introduce useENSAvatar hook.
Mar 23 2023, 9:19 PM
ashoat requested review of D7164: [keyserver] Launch NSE for build 199 and above.
Mar 23 2023, 8:47 PM
ashoat requested review of D7163: [native] Introduce useNativeMessageListData and NativeChatMessageItem.
Mar 23 2023, 8:36 PM
ashoat closed D7093: [native] Use a JSON blob for heightMeasurerKey.
Mar 23 2023, 8:17 PM
ashoat committed rCOMM4765c224e781: [native] Use a JSON blob for heightMeasurerKey (authored by ashoat).
[native] Use a JSON blob for heightMeasurerKey
Mar 23 2023, 8:17 PM
ashoat requested changes to D7162: [keyserver] Update `fetchUserInfos` to include `avatar`s in `GlobalUserInfo`s.

That makes sense RE older clients. Thinking about it more, I'm not actually sure why we ever thought it made sense to "hide" the avatar field from older clients... they're going to eventually need the field, and if we hide it from them initially then they will need to rely on the state check mechanism to fix things after they upgrade codeVersions.

Mar 23 2023, 8:06 PM
ashoat added a comment to D6924: [keyserver] Introduce columns in messages table to support pinned messages.

The current fetchPinnedMessageInfos query doesn't have the ability to paginate, but it can always be added in later. We could initially fetch all of the pinned messages at once from the server, but have a task to paginate the fetching.

Mar 23 2023, 7:46 PM
ashoat accepted D7110: [keyserver/lib] Add a fetchPinnedMessages method to retrieve all pinned messages for a thread.
Mar 23 2023, 7:44 PM
ashoat added a comment to D5995: [native] clear database when malformation is detected.

Yeah we can probably get rid of them, feel free to put up a diff

Mar 23 2023, 7:37 PM
ashoat accepted D7156: [lib] Update `CurrentUserInfo` on `updateUserAvatarActionTypes.success`.
Mar 23 2023, 7:37 PM
ashoat accepted D7157: [keyserver] `JSON.parse()` stringified `avatar` in `fetchLoggedInUserInfo`.
Mar 23 2023, 7:35 PM
ashoat added a comment to D7067: [native] render user avatars in chat screen.

One last thing that would be great if you could test: what happens if you upload a single very "thin" photo? I want to make sure it doesn't expand to take the whole width and then becomes super tall. (Or at least, that there's no regression in this.)

Mar 23 2023, 7:34 PM
ashoat accepted D7067: [native] render user avatars in chat screen.
Mar 23 2023, 7:32 PM
ashoat requested changes to D7141: [keyserver] Display edited message in the sidebar if the source message is edited.

Back to your queue for nits and fetchDerivedMessages

Mar 23 2023, 7:26 PM
ashoat added inline comments to D7148: [keyserver/lib] Introduce a new message type and spec for toggling pins.
Mar 23 2023, 7:26 PM
ashoat added inline comments to D7148: [keyserver/lib] Introduce a new message type and spec for toggling pins.
Mar 23 2023, 7:25 PM
ashoat added a reviewer for D7148: [keyserver/lib] Introduce a new message type and spec for toggling pins: kuba.

Adding @kuba here – he is looking at fetchDerivedMessages right now in D7141

Mar 23 2023, 7:25 PM
ashoat added a reviewer for D7141: [keyserver] Display edited message in the sidebar if the source message is edited: rohan.
Mar 23 2023, 7:24 PM
ashoat added inline comments to D7120: Use notifications permission request API on Android 13 while keeping old behavior on older Androids..
Mar 23 2023, 7:17 PM
ashoat added inline comments to D7106: Native Java code to request notifications permissions from JS on Android 13.
Mar 23 2023, 7:13 PM
ashoat attached a referenced file: F442421: Screenshot 2023-03-23 at 9.53.39 PM.png.
Mar 23 2023, 6:59 PM
ashoat requested changes to D7105: Build files set-up for Android 13.

Can you please amend the Test Plan to include:

Mar 23 2023, 6:59 PM
ashoat accepted D7161: [ios] Add `--repo-update` to `pod install` command.
Mar 23 2023, 2:05 PM
ashoat accepted D7154: [web-db] get client database store from SQLite.
Mar 23 2023, 12:55 PM
ashoat added inline comments to D7101: [web-db] add `sql.js` types.
Mar 23 2023, 12:54 PM
ashoat accepted D7153: [web-db] process Draft Store Operations on worker.
Mar 23 2023, 12:52 PM
ashoat accepted D7124: [web-db] add unit tests for draft store queries.

Did not review this closely

Mar 23 2023, 12:50 PM
ashoat accepted D7123: [web-db] implement function get all drafts from SQLite.
Mar 23 2023, 12:48 PM
ashoat added inline comments to D7118: [web-db] add function for parsing SQLite query.
Mar 23 2023, 12:46 PM
ashoat added inline comments to D7118: [web-db] add function for parsing SQLite query.
Mar 23 2023, 12:45 PM
ashoat accepted D7121: [web-db] create drafts table in SQLite database.

I'm sorry, I completely missed that it was an old migration. My apologies for making assumptions and leaving an unnecessarily aggressive comment

Mar 23 2023, 12:39 PM
ashoat accepted D7118: [web-db] add function for parsing SQLite query.
Mar 23 2023, 12:38 PM
ashoat accepted D7101: [web-db] add `sql.js` types.
Mar 23 2023, 12:37 PM
ashoat added a comment to D6978: [web] Add a badge on the Inbox button.

On context on why this diff has been up for 2 weeks and hasn't been reviewed?

Mar 23 2023, 12:30 PM
ashoat closed D7150: [web] Scaffolding for opaque-ke-wasm project.
Mar 23 2023, 12:20 PM
ashoat committed rCOMMede918e1a6f5: [web] Scaffolding for opaque-ke-wasm project (authored by ashoat).
[web] Scaffolding for opaque-ke-wasm project
Mar 23 2023, 12:20 PM
ashoat closed D7149: [web] Add grpc-web@1.4.2.
Mar 23 2023, 12:20 PM
ashoat committed rCOMM4559bf844167: [web] Add grpc-web@1.4.2 (authored by ashoat).
[web] Add grpc-web@1.4.2
Mar 23 2023, 12:20 PM
ashoat added a comment to D7150: [web] Scaffolding for opaque-ke-wasm project.

Note that I didn't bother adding a line here because @commapp/opaque-ke-wasm isn't actually consumed directly via Yarn workspaces, so we don't need its package.json in order to run yarn cleaninstall. Instead, we'll consume published versions of @commapp/opaque-ke-wasm as an NPM package.

Mar 23 2023, 12:12 PM
ashoat added inline comments to D7083: Amended Account & Session types..
Mar 23 2023, 8:02 AM
ashoat closed D7151: [native] Migrate from react-native-fast-image to expo-image.
Mar 23 2023, 5:27 AM
ashoat committed rCOMM115e490e58c1: [native] Migrate from react-native-fast-image to expo-image (authored by ashoat).
[native] Migrate from react-native-fast-image to expo-image
Mar 23 2023, 5:27 AM
ashoat added a comment to D7118: [web-db] add function for parsing SQLite query.

The problem is that we would need to validate each query result manually, going through each column one-by-one and checking the result type. This isn’t something we do today for MySQL queries typically. If we feel strongly about doing input validation for SQL query results we could do with tcomb, but I’m not sure that the effort is worth it.

Mar 23 2023, 5:12 AM

Mar 22 2023

ashoat updated the test plan for D7151: [native] Migrate from react-native-fast-image to expo-image.
Mar 22 2023, 9:16 PM
ashoat added a comment to D7151: [native] Migrate from react-native-fast-image to expo-image.

Just tested and it looks like this solves rendering SVG data URIs for us!! I haven't tested other types, but PNG, JPEG, and SVG are all listed as supported in the docs. Thanks @bartek for the tip! This is way better than what I had planned to do on Android.

Mar 22 2023, 9:15 PM
ashoat added a comment to D7151: [native] Migrate from react-native-fast-image to expo-image.

Done testing!! I think this is safe to land. I haven't tested blurhash or SVG / data URI support yet, but those are things that react-native-fast-image can't handle today anyways

Mar 22 2023, 8:29 PM
ashoat updated the test plan for D7151: [native] Migrate from react-native-fast-image to expo-image.
Mar 22 2023, 8:28 PM
ashoat updated the test plan for D7151: [native] Migrate from react-native-fast-image to expo-image.
Mar 22 2023, 5:58 PM
ashoat requested review of D7151: [native] Migrate from react-native-fast-image to expo-image.
Mar 22 2023, 5:28 PM
ashoat requested review of D7149: [web] Add grpc-web@1.4.2.
Mar 22 2023, 4:42 PM
ashoat requested review of D7150: [web] Scaffolding for opaque-ke-wasm project.
Mar 22 2023, 4:42 PM
ashoat requested changes to D7067: [native] render user avatars in chat screen.

That general approach looks great!! Great work figuring this out.

Mar 22 2023, 4:09 PM
ashoat accepted D7115: [Identity] Update X3DH protobuf to differiente key requests.
Mar 22 2023, 3:51 PM
ashoat added inline comments to D7106: Native Java code to request notifications permissions from JS on Android 13.
Mar 22 2023, 3:41 PM
ashoat requested changes to D7148: [keyserver/lib] Introduce a new message type and spec for toggling pins.
Mar 22 2023, 3:37 PM
ashoat added inline comments to D6924: [keyserver] Introduce columns in messages table to support pinned messages.
Mar 22 2023, 3:27 PM
ashoat added inline comments to D7110: [keyserver/lib] Add a fetchPinnedMessages method to retrieve all pinned messages for a thread.
Mar 22 2023, 3:26 PM
ashoat accepted D7145: [keyserver] Adding edit_message to the sidebar thread.
Mar 22 2023, 3:25 PM
ashoat accepted D7144: [lib] Added edit message action reducer.
Mar 22 2023, 3:22 PM
ashoat added inline comments to D7129: [lib] Added new 'sendEditMessage' action.
Mar 22 2023, 3:16 PM
ashoat added inline comments to D7141: [keyserver] Display edited message in the sidebar if the source message is edited.
Mar 22 2023, 3:15 PM
ashoat requested changes to D7143: [lib] Function to dispatch edit message action.
Mar 22 2023, 3:14 PM
ashoat added inline comments to D7140: [lib] Displaying edited content in a pending thread.
Mar 22 2023, 3:11 PM
ashoat requested changes to D7141: [keyserver] Display edited message in the sidebar if the source message is edited.
Mar 22 2023, 3:08 PM
ashoat requested changes to D7139: [lib] Added method to call edit_message endpoint.
Mar 22 2023, 3:02 PM
ashoat requested changes to D7121: [web-db] create drafts table in SQLite database.
Mar 22 2023, 2:59 PM
ashoat requested changes to D7118: [web-db] add function for parsing SQLite query.

We should not be polluting the global namespace in D7101. You should not be able to use QueryExecResult without importing it unless it's in the Flow lib.

Mar 22 2023, 2:53 PM
ashoat requested changes to D7101: [web-db] add `sql.js` types.
Mar 22 2023, 2:49 PM
ashoat added a comment to D6965: [lib][native] Displaying edited messages from database.

We can consider preventing users from sending empty edit messages.

Mar 22 2023, 2:46 PM
ashoat added a comment to D7094: [keyserver] Send updateBadgeCount notifs to macOS.

This diff seems to call getAPNsNotificationTopic with two parameters, but the version of that function in master only takes one parameter. Can you please update this diff's dependencies so I can see the rest of your stack? Please always set diff dependencies!

There's no stack for this diff, I'm calling getAPNsNotificationTopic with one argument, it's just an object with two fields.

Mar 22 2023, 2:08 PM
ashoat added inline comments to D6961: [lib] Create types for message editing.
Mar 22 2023, 2:07 PM
ashoat added a comment to D7065: [native] Added displaying labels to InlineEngagement.

Will this affect the height of the item? Have you made the necessary changes to the height determination code (textMessageItemHeight)?

Mar 22 2023, 2:04 PM
ashoat requested changes to D7117: [keyserver] Add migration - process messages in our db for search.
Mar 22 2023, 2:01 PM
ashoat accepted D7138: [native] render the rest of the thread avatars.
Mar 22 2023, 1:56 PM
ashoat requested changes to D7137: [native] render thread avatars for parent/child threads in thread settings.
Mar 22 2023, 1:51 PM
ashoat requested changes to D7136: [native] render thread avatars in community drawer.

You should definitely have included @inka on this review – she is by far the expert on this code

Mar 22 2023, 12:42 PM
ashoat accepted D7135: [native] render thread avatar in CHANNEL AVATAR thread settings section.
Mar 22 2023, 12:29 PM
ashoat requested changes to D7134: [native] update message list header to new design.
Mar 22 2023, 12:27 PM
ashoat requested changes to D7133: [lib] introduce getAvatarForThreadEntity function.

I am skeptical that you need to be touching entity-text.js or entity-helpers.js at all. Can you explain what updates you think you need to the EntityText framework and why?

Mar 22 2023, 12:22 PM
ashoat accepted D7131: [lib] introduce getDefaultAvatar helper function.
Mar 22 2023, 12:13 PM
ashoat requested changes to D7132: [lib] introduce useGetAvatarForThread hook.

Mostly questions

Mar 22 2023, 12:12 PM
ashoat added inline comments to D7129: [lib] Added new 'sendEditMessage' action.
Mar 22 2023, 11:59 AM
ashoat accepted D7127: [lib][keyserver] Introduce new threadPermissions types for editing messages.
Mar 22 2023, 11:46 AM
ashoat requested changes to D7120: Use notifications permission request API on Android 13 while keeping old behavior on older Androids..

Just a question

Mar 22 2023, 11:42 AM
ashoat accepted D7105: Build files set-up for Android 13.

Please update native/expo-modules/aes-crypto/android/build.gradle before landing!

Mar 22 2023, 11:39 AM
ashoat added inline comments to D7104: [web-db] add unit tests for worker crypto utils.
Mar 22 2023, 11:37 AM
ashoat accepted D7122: [web-db] implement Draft Store Operations.
Mar 22 2023, 11:36 AM
ashoat accepted D7126: [lib] Introduce `updateUserAvatar` to `user-actions`.
Mar 22 2023, 11:32 AM
ashoat requested changes to D7106: Native Java code to request notifications permissions from JS on Android 13.
Mar 22 2023, 11:28 AM
ashoat requested changes to D7125: Use new notifications permission API to modify push notifs settings alerts on Android 13.
Mar 22 2023, 11:23 AM
ashoat requested changes to D7110: [keyserver/lib] Add a fetchPinnedMessages method to retrieve all pinned messages for a thread.
Mar 22 2023, 8:36 AM