Page MenuHomePhabricator
Feed Advanced Search

Jun 2 2023

michal updated the diff for D8013: [web] Use search users in chat composer.

Removed code duplication (function was introduced in D8010)

Jun 2 2023, 5:40 AM
michal updated the diff for D8012: [keyserver] Include initial navInfo selectedUserList.

Comment and readonly

Jun 2 2023, 5:38 AM
michal requested review of D8010: [native] Use search users in chat composer.
Jun 2 2023, 5:33 AM
michal updated the diff for D8010: [native] Use search users in chat composer.

Extracted the shared code (here and not in D8013, because that's how I have my git stack set up).

Jun 2 2023, 5:32 AM
michal added inline comments to D8009: [lib] Include searchUsers results in getPotentialMemberItems.
Jun 2 2023, 5:21 AM
michal updated the diff for D8009: [lib] Include searchUsers results in getPotentialMemberItems.

Change the function to take a object of params, fix

Jun 2 2023, 5:21 AM

May 30 2023

michal accepted D7986: [web-db] dispatch reports from SQLite to store.
May 30 2023, 8:27 AM
michal accepted D7985: [web-db] start processing report store operations.
May 30 2023, 8:27 AM
michal accepted D7984: [web-db] add report table on web.
May 30 2023, 8:27 AM
michal accepted D7983: [web-db] implement SQL queries for report store.
May 30 2023, 8:26 AM
michal accepted D7982: [web-db] add migration logic to SQLite on web.

I think you could test the migration code by mocking the migrations variable with jest, but not sure if that's worth it.

May 30 2023, 8:26 AM
michal accepted D7977: [native] migrate reports to SQLite.
May 30 2023, 8:25 AM
michal accepted D7975: [CommCoreModule] implement synchronous report store operations.
May 30 2023, 8:25 AM

May 29 2023

michal requested review of D8014: [web] Revert wordmark position on macos.
May 29 2023, 7:37 AM
michal requested review of D8013: [web] Use search users in chat composer.
May 29 2023, 7:18 AM
michal requested review of D8012: [keyserver] Include initial navInfo selectedUserList.
May 29 2023, 7:16 AM
michal requested review of D8011: [web] Change navInfo.selectedUserList to contains user infos.
May 29 2023, 7:10 AM
michal requested review of D8010: [native] Use search users in chat composer.
May 29 2023, 6:59 AM
michal requested review of D8009: [lib] Include searchUsers results in getPotentialMemberItems.
May 29 2023, 6:23 AM

May 25 2023

michal accepted D7971: [web] Disabled edit button if edited message is empty.
May 25 2023, 4:31 AM
michal accepted D7955: [native] Change border radius of filled-in input fields.
May 25 2023, 2:05 AM
michal accepted D7954: [web] Change border radius of filled-in input fields.
May 25 2023, 2:05 AM
michal accepted D7970: [web] Change edit icon to filled.
May 25 2023, 2:02 AM

May 24 2023

michal accepted D7837: [web] Added FocusTrap for modals.
May 24 2023, 2:47 AM

May 23 2023

michal accepted D7941: [keyserver] Convert flag to a boolean.
May 23 2023, 7:12 AM
michal accepted D7823: [web] Added EditTextMessage component.
May 23 2023, 6:15 AM
michal added a comment to D7837: [web] Added FocusTrap for modals.

Could we just add it to ModalOverlay so it applies to every modal automatically?

May 23 2023, 4:41 AM
michal accepted D7836: [web] Displaying modal of editing message GUI.
May 23 2023, 4:32 AM
michal accepted D7833: [web] Storing edited message component position.
May 23 2023, 4:29 AM
michal accepted D7823: [web] Added EditTextMessage component.
May 23 2023, 4:26 AM
michal added a comment to D7903: [keyserver] Don't use web version when sending desktop notifs.

Are you sure about this? Don't we need both web (js code) version and desktop (electron)?

May 23 2023, 3:45 AM
michal accepted D6968: [web] Add to redux state picked community in the Chat tab.

Lgtm

May 23 2023, 12:33 AM

May 22 2023

michal accepted D7849: [web] Prevent user from closing the tab with unsaved edits.
May 22 2023, 6:16 AM
michal requested review of D7903: [keyserver] Don't use web version when sending desktop notifs.
May 22 2023, 5:44 AM
michal added a comment to D7854: [keyserver] Handle web version in hasMinCodeVersion.

Do we still have places where we don't use hasMinCodeVersion function? Do we have to update these?

May 22 2023, 5:27 AM
michal updated the diff for D7851: [lib] Move id conversion functions to lib.

Also move tests for id conversion to lib

May 22 2023, 4:35 AM

May 19 2023

michal added inline comments to D7852: [web] Convert ids during state check.
May 19 2023, 3:41 AM
michal requested review of D7855: [web] Send platform details on websocket connection.
May 19 2023, 3:40 AM
michal added inline comments to D7836: [web] Displaying modal of editing message GUI.
May 19 2023, 3:10 AM
michal added inline comments to D7823: [web] Added EditTextMessage component.
May 19 2023, 3:04 AM
michal added inline comments to D7833: [web] Storing edited message component position.
May 19 2023, 3:01 AM
michal accepted D7828: [web] Added message editing.
May 19 2023, 2:56 AM
michal accepted D7794: [web] Add new edit message provider.
May 19 2023, 2:55 AM

May 18 2023

michal added a comment to D7830: [web] Added setting `Edit failed` state.

I think you could merge this one too. Thanks!

May 18 2023, 5:35 AM
michal accepted D7827: [web] Display edit state (in the background).
May 18 2023, 4:16 AM
michal accepted D7825: [web] Store edited message draft in the state.
May 18 2023, 4:16 AM
michal accepted D7850: [web] Handle new messages when in edit mode.
May 18 2023, 4:16 AM
michal added inline comments to D7849: [web] Prevent user from closing the tab with unsaved edits.
May 18 2023, 4:09 AM
michal added a comment to D7837: [web] Added FocusTrap for modals.

Not a fan of this from an accessibility stand point. I found something like this: focus-trap, we could wrap all our modals with it. But I'm not sure if that's worth it. @ashoat any thoughts?

May 18 2023, 4:00 AM
michal accepted D7793: [web] Added edit button to the Tooltip.

Accepting, but please fix the icon before landing!

May 18 2023, 3:48 AM

May 17 2023

michal added inline comments to D7855: [web] Send platform details on websocket connection.
May 17 2023, 10:44 AM
michal requested changes to D7823: [web] Added EditTextMessage component.

Just a few more points about the buttons, otherwise looks good!

May 17 2023, 7:09 AM
michal added a comment to D7793: [web] Added edit button to the Tooltip.

As far as I know, diffs should be small, and do only one thing. The division between those diffs is simple: one diff adds the button itself to the tooltip, and the other adds its functionality (entering edit state).
Also, this diff cannot depend on previous diffs, because it is first in the stack.

May 17 2023, 6:29 AM
michal requested changes to D7823: [web] Added EditTextMessage component.
May 17 2023, 5:59 AM
michal requested review of D7855: [web] Send platform details on websocket connection.
May 17 2023, 5:49 AM
michal requested review of D7854: [keyserver] Handle web version in hasMinCodeVersion.
May 17 2023, 5:44 AM
michal requested review of D7852: [web] Convert ids during state check.
May 17 2023, 5:27 AM
michal requested review of D7851: [lib] Move id conversion functions to lib.
May 17 2023, 5:25 AM
michal requested review of D7808: [keyserver] Convert upload responders.
May 17 2023, 3:50 AM
michal updated the diff for D7808: [keyserver] Convert upload responders.

Rebase, validate uploadMediaMetadataResponder, fix bug with uploadDownloadResponder

May 17 2023, 3:42 AM
michal requested review of D7848: [keyserver] Validate fetchPrimaryInviteLinksResponder output.
May 17 2023, 2:50 AM
michal added a comment to D7807: [keyserver] Validate link responders.

Sorry, I should have noticed that during rebase, I will put up a diff with the validators.

May 17 2023, 1:55 AM

May 16 2023

michal requested changes to D7793: [web] Added edit button to the Tooltip.

These //TODO comments make it hard to review this stack because now the diffs depend on both previous and future diffs. This diff in particular seem like it would be easy to merge with D7824. Is there a reason why you did it this way?

May 16 2023, 11:05 AM
michal added inline comments to D7794: [web] Add new edit message provider.
May 16 2023, 11:03 AM
michal requested changes to D7823: [web] Added EditTextMessage component.
May 16 2023, 11:01 AM
michal accepted D7822: [web] Added ChatInputTextArea component.
May 16 2023, 11:01 AM
michal added inline comments to D7808: [keyserver] Convert upload responders.
May 16 2023, 9:47 AM
michal updated the diff for D7805: [keyserver] Use platform details in validateOutput.

Use FUTURE_CODE_VERSION

May 16 2023, 9:36 AM

May 15 2023

michal requested review of D7808: [keyserver] Convert upload responders.
May 15 2023, 4:15 AM
michal requested review of D7807: [keyserver] Validate link responders.
May 15 2023, 4:12 AM
michal requested review of D7806: [keyserver] Validate notifications.
May 15 2023, 4:08 AM
michal requested review of D7805: [keyserver] Use platform details in validateOutput.
May 15 2023, 4:06 AM
michal accepted D7754: [lib] modify `assertObjectsAreEqual` to return deep differential instead of only one level keys.
May 15 2023, 4:01 AM
michal updated the diff for D7783: [lib] Split thread-types.

Add newline

May 15 2023, 3:45 AM
michal updated the diff for D7749: [keyserver] Validate websocket output.
  • Validate also when viewer is null (when error socket message is sent)
  • APIResponseServerSocketMessage can have no payload (for example when calling deviceTokenUpdateResponder which returns void). Updated the socket type and the validator to match this
May 15 2023, 3:42 AM

May 11 2023

michal requested review of D7786: [web] Validate website responder.
May 11 2023, 2:35 AM
michal requested review of D7788: [lib] Move update type enum into a separate file.
May 11 2023, 2:34 AM
michal accepted D7787: [web-db] stop supporting Safari.
May 11 2023, 2:14 AM

May 10 2023

michal requested review of D7785: [lib] Move socket errors to errors.js.
May 10 2023, 10:33 AM
michal updated the diff for D7783: [lib] Split thread-types.

Remove unnecessary changes

May 10 2023, 10:10 AM
michal requested review of D7783: [lib] Split thread-types.
May 10 2023, 10:07 AM
michal requested review of D7750: [keyserver] Convert websocket input.

Are you sure this is correct? Shouldn't we also convert a response from the server?

If I'm not misunderstanding, the responses are validated in D7749.

May 10 2023, 6:57 AM

May 9 2023

michal added inline comments to D7753: [lib] introduce function to find deep difference between two objects.
May 9 2023, 5:56 AM
michal retitled D7745: [lib] Fix circular dependency from [lib] Extract regexes from account-utils to [lib] Fix circular dependency.
May 9 2023, 2:22 AM
michal updated the diff for D7745: [lib] Fix circular dependency.

Instead of extracting the regexes, instead move the session functions. Only invalidSessionRecovery had to be moved but I moved both of them to keep them together, I can move invalidSessionDowngrade back if that's prefered.

May 9 2023, 2:22 AM
michal requested review of D7750: [keyserver] Convert websocket input.
May 9 2023, 2:01 AM
michal requested review of D7749: [keyserver] Validate websocket output.
May 9 2023, 1:54 AM

May 8 2023

michal requested review of D7745: [lib] Fix circular dependency.
May 8 2023, 10:49 AM
michal added inline comments to D7713: [keyserver] Mark ids in input validators.
May 8 2023, 3:22 AM
michal updated the test plan for D7713: [keyserver] Mark ids in input validators.
May 8 2023, 1:43 AM
michal updated the diff for D7713: [keyserver] Mark ids in input validators.
  • Updated the test plan
  • Also converted role ids. This required the conversion logic to handle tComb refinements, so I added that and tests
May 8 2023, 1:41 AM
michal added inline comments to D7711: [keyserver] Introduce validateOutput function.
May 8 2023, 1:20 AM
michal updated the diff for D7711: [keyserver] Introduce validateOutput function.

Changed the output validation from server error to a warning. This will be temporary, but with this we can safely check for validator <-> flow-types/runtime-objects mismatches without breaking clients. We can change it back once we activate the conversion logic. Included the id in the error message of serverID -> clientID conversion.

May 8 2023, 1:18 AM

May 5 2023

michal added a comment to D7714: [keyserver] Introduce input id conversion.

Sorry should have mentioned this. There are added because flow types and validators are not in sync. These properties were removed at one point and aren't used. They still exist in the validators so we can support older clients.
Now that tShape<T> is checking if properties count and names match between the flow type and the validator we get an error. Adding empty makes them match, while still blocks the rest of the code from using these values, because they are empty.

May 5 2023, 4:21 AM
michal added a comment to D7710: [keyserver] Introduce report, message and thread responder validators.

I've talked with @ashoat about report ids, and in the future, we probably will send them to some centralized server, so we don't need to convert ids or payloads. Currently the ids aren't used anywhere on the client side (except for being displayed in an alert once).

May 5 2023, 4:16 AM
michal updated the diff for D7710: [keyserver] Introduce report, message and thread responder validators.

Fix one test that was being skipped by mistake

May 5 2023, 4:15 AM
michal updated the diff for D7569: [lib] Introduce raw thread info validators.

Also convert roles

May 5 2023, 4:11 AM
michal updated the diff for D7567: [lib] Introduce raw message info validators.

Also convert roles

May 5 2023, 4:06 AM
michal updated the diff for D7566: [lib] Introduce media validators.

Also convert media ids

May 5 2023, 3:56 AM
michal added inline comments to D7711: [keyserver] Introduce validateOutput function.
May 5 2023, 3:25 AM