Page MenuHomePhabricator
Feed Advanced Search

Dec 29 2022

michal requested review of D6090: [native] Support rainbow url scheme for Android.
Dec 29 2022, 4:23 AM

Dec 23 2022

michal added inline comments to D5882: [desktop] Base windows config.
Dec 23 2022, 5:35 AM
michal updated the diff for D5882: [desktop] Base windows config.

Simplified code.
Task for the CI build: ENG-2532

Dec 23 2022, 5:34 AM

Dec 22 2022

michal accepted D5984: [web] Remove timeZone from Redux state.
Dec 22 2022, 10:37 AM
michal accepted D5982: [keyserver] Fix JSONStream types.
Dec 22 2022, 10:34 AM
michal accepted D5983: [web] Only render Loading component in SSR.
Dec 22 2022, 10:32 AM
michal accepted D5985: [web] Remove userAgent from Redux state.
Dec 22 2022, 10:27 AM
michal accepted D5981: [web] Introduce Loading component to web.
Dec 22 2022, 10:23 AM

Dec 20 2022

michal updated the summary of D5882: [desktop] Base windows config.
Dec 20 2022, 7:18 AM

Dec 19 2022

michal requested review of D5882: [desktop] Base windows config.
Dec 19 2022, 5:54 AM
michal attached a referenced file: F287208: image.png.
Dec 19 2022, 5:37 AM
michal planned changes to D5882: [desktop] Base windows config.
Dec 19 2022, 4:56 AM
michal added inline comments to D5882: [desktop] Base windows config.
Dec 19 2022, 4:44 AM
michal updated the diff for D5882: [desktop] Base windows config.

Rewrote the Squirrel startup logic a bit.

Dec 19 2022, 4:41 AM

Dec 16 2022

michal added inline comments to D5882: [desktop] Base windows config.
Dec 16 2022, 7:21 AM
michal requested review of D5882: [desktop] Base windows config.
Dec 16 2022, 7:19 AM
michal updated the diff for D5869: [desktop] Display a warning when trying to build macos universal.

Use yarn clean-build in yarn clean

Dec 16 2022, 6:53 AM

Dec 15 2022

michal added a comment to D5868: [desktop] Fix error when trying to maximize after closing the window.

Yeah

Dec 15 2022, 11:13 AM
michal added a comment to D5869: [desktop] Display a warning when trying to build macos universal.

Looks good, but does Windows still need rimraf (from this package) instead of rm -rf? Or do a modern Windows environment handle rm -rf well?

It doesn't handle the -rf unfortunately, but I'm not sure if it's worth it thinking about now. This fix is only required for the macOS builds and there are a lot of other things that won't work on windows (like using && in commands or ENV=dev). If we want all the scripts in desktop to work on windows we should probably use some more general solution (maybe something like this? but I'm not sure about it).

Dec 15 2022, 1:27 AM

Dec 14 2022

michal requested review of D5869: [desktop] Display a warning when trying to build macos universal.
Dec 14 2022, 10:53 AM
michal requested review of D5868: [desktop] Fix error when trying to maximize after closing the window.
Dec 14 2022, 10:36 AM

Dec 7 2022

michal updated the diff for D5825: [desktop] Code signing for disk image.

Remove the @electron-forge/maker-zip dependency.

Dec 7 2022, 6:49 AM
michal added inline comments to D5735: [desktop] Add babel support.
Dec 7 2022, 5:41 AM
michal updated the diff for D5735: [desktop] Add babel support.

Fix typo and update versions

Dec 7 2022, 5:41 AM
michal updated the diff for D5825: [desktop] Code signing for disk image.

The bundleID wasn't completely intentional, thanks for noticing. I don't think it would cause problems (apple offers universal purchase which requires the same ID on both iOS and macOS, and we could create a macOS section in the current comm App Store Record), but I don't think there are any advantages for us so I'm changes it to app.comm.macos.

Dec 7 2022, 2:19 AM

Dec 6 2022

michal requested review of D5825: [desktop] Code signing for disk image.
Dec 6 2022, 7:43 AM
michal updated the diff for D5735: [desktop] Add babel support.

Small changes: add name to the BabelPlugin (it's more clearer what's happening in the stdout during packaging) and simplify ignore pattern to ignore whole folder.

Dec 6 2022, 6:42 AM
michal updated the diff for D5735: [desktop] Add babel support.

Add comments, change order in .eslintignore

Dec 6 2022, 5:45 AM
michal added a comment to D5735: [desktop] Add babel support.

Sorry, I sent the comments by mistake before finishing the inline comment about nohoist and I don't think I can edit it.

Dec 6 2022, 5:36 AM
michal added a comment to D5735: [desktop] Add babel support.

electron-forge is a package that handles building, packaging, and code signing. It uses different packages for this (e.g. electron-packager, osx-sign) and provides a unified configuration for them. It's not included in the final application. When you run electron-forge package or some other command it uses forge.config.cjs for configuration.

Dec 6 2022, 5:29 AM

Dec 5 2022

michal requested review of D5735: [desktop] Add babel support.
Dec 5 2022, 4:02 AM

Nov 22 2022

michal updated the diff for D5629: [desktop] Basic electron app.

Add --logo-bg css variable.

Nov 22 2022, 3:18 AM

Nov 21 2022

michal updated the diff for D5632: [desktop] Draggable window.

Css fixes

Nov 21 2022, 3:55 AM
michal updated the diff for D5631: [desktop] Double-click to maximize or minimize.

Add a comment

Nov 21 2022, 2:56 AM
michal updated the diff for D5630: [desktop] Navigation arrows.

Fix issues

Nov 21 2022, 2:42 AM
michal attached a referenced file: F254592: image.png.
Nov 21 2022, 2:34 AM
michal attached a referenced file: F254594: image.png.
Nov 21 2022, 2:34 AM
michal requested review of D5629: [desktop] Basic electron app.
Nov 21 2022, 2:34 AM
michal updated the diff for D5629: [desktop] Basic electron app.

Small changes.

Nov 21 2022, 2:33 AM

Nov 17 2022

michal added inline comments to D5633: [desktop] Notification badge.
Nov 17 2022, 2:15 PM
michal updated the diff for D5633: [desktop] Notification badge.

Rebase, small fixes

Nov 17 2022, 2:15 PM
michal added inline comments to D5632: [desktop] Draggable window.
Nov 17 2022, 2:07 PM
michal updated the diff for D5632: [desktop] Draggable window.

Moved the new css class to the header.

Nov 17 2022, 2:05 PM
michal updated the diff for D5631: [desktop] Double-click to maximize or minimize.

Don't define arrow functions on every render, move the doubleClick callbacks to the <header>.

Nov 17 2022, 1:58 PM
michal added inline comments to D5630: [desktop] Navigation arrows.
Nov 17 2022, 1:53 PM
michal updated the diff for D5630: [desktop] Navigation arrows.

This revision also fixes two additional issues that I haven't noticed before:

  • the onNavigate callback should be set in useEffect, and should be removed in the useEffect returned callback
  • the history should be cleared when user logs in/out. Otherwise the navigation arrows behave incorrectly after relogging.
Nov 17 2022, 1:51 PM
michal updated the diff for D5629: [desktop] Basic electron app.

Revert formatting. Sorry, I have format-on-save enabled so the changes were made automatically, but I should have noticed that.

Nov 17 2022, 1:14 PM
michal updated the diff for D5629: [desktop] Basic electron app.

Fix keyserver build

Nov 17 2022, 1:02 PM
michal added inline comments to D5629: [desktop] Basic electron app.
Nov 17 2022, 10:31 AM
michal added a comment to D5629: [desktop] Basic electron app.

Here's the linear webpack issue, I'm going to focus on it whenever I have the time.

Nov 17 2022, 9:39 AM
michal updated the diff for D5629: [desktop] Basic electron app.

Rename to .cjs, documentation changes and other fixes

Nov 17 2022, 9:31 AM

Nov 16 2022

michal updated the diff for D5630: [desktop] Navigation arrows.

Fix small issues

Nov 16 2022, 9:56 AM
michal updated the diff for D5629: [desktop] Basic electron app.

Responded to some of the review, left comments for the rest

Nov 16 2022, 9:01 AM
michal added inline comments to D5629: [desktop] Basic electron app.
Nov 16 2022, 6:43 AM
michal added inline comments to D5633: [desktop] Notification badge.
Nov 16 2022, 6:28 AM
michal added inline comments to D5632: [desktop] Draggable window.
Nov 16 2022, 6:27 AM
michal added inline comments to D5630: [desktop] Navigation arrows.
Nov 16 2022, 6:24 AM
michal added a comment to D5629: [desktop] Basic electron app.

@tomek @ashoat The problems with ESM, and flow typings are connected.

Nov 16 2022, 6:17 AM

Nov 15 2022

michal requested review of D5633: [desktop] Notification badge.
Nov 15 2022, 1:05 AM
michal requested review of D5632: [desktop] Draggable window.
Nov 15 2022, 1:02 AM
michal requested review of D5631: [desktop] Double-click to maximize or minimize.
Nov 15 2022, 12:54 AM
michal requested review of D5630: [desktop] Navigation arrows.
Nov 15 2022, 12:38 AM
michal added inline comments to D5629: [desktop] Basic electron app.
Nov 15 2022, 12:25 AM
michal requested review of D5629: [desktop] Basic electron app.
Nov 15 2022, 12:14 AM

Nov 9 2022

michal requested review of D5583: [web] Change alert type in UserListItem.
Nov 9 2022, 5:56 AM
michal updated the diff for D5518: [web] Unify behaviour of chat composer with native.

Split into two diffs

Nov 9 2022, 5:42 AM
michal requested review of D5463: [web] Call searchUsers in chat composer.
Nov 9 2022, 3:37 AM
michal added inline comments to D5463: [web] Call searchUsers in chat composer.
Nov 9 2022, 3:25 AM

Nov 7 2022

michal requested review of D5518: [web] Unify behaviour of chat composer with native.
Nov 7 2022, 4:07 AM
michal updated the diff for D5518: [web] Unify behaviour of chat composer with native.

Changed type, changed the definition of the UserListItem.

Nov 7 2022, 4:07 AM
michal added a comment to D5463: [web] Call searchUsers in chat composer.

Some additional notes in the inline comments.

Nov 7 2022, 3:40 AM
michal added inline comments to D5463: [web] Call searchUsers in chat composer.
Nov 7 2022, 3:23 AM
michal updated the diff for D5463: [web] Call searchUsers in chat composer.

Update

Nov 7 2022, 3:23 AM
michal updated the summary of D5518: [web] Unify behaviour of chat composer with native.
Nov 7 2022, 1:19 AM

Nov 2 2022

michal requested review of D5518: [web] Unify behaviour of chat composer with native.
Nov 2 2022, 10:14 AM
michal updated the diff for D5463: [web] Call searchUsers in chat composer.
  • New task for server-side checks created here
  • Reverted changes to the forceAddMembers in the keyserver
  • Unifying behaviour between native and web app UI when it comes to the selecting non-friend users will be included in the following diff
Nov 2 2022, 9:53 AM
michal updated the summary of D5463: [web] Call searchUsers in chat composer.
Nov 2 2022, 9:49 AM
michal added a comment to D5463: [web] Call searchUsers in chat composer.

@ashoat the native app currently works like this:

  • if you try to create a chat with someone you are not friends with, the app exits search and puts you in a 1-1 thread with this user
  • if you already have someone selected and you try to add a non-friend the app displays an alert
Nov 2 2022, 5:20 AM
michal updated the diff for D5462: [keyserver] Update userStore from chat creation url.

Don't create the relationship

Nov 2 2022, 4:15 AM
michal updated the diff for D5472: [web] Calendar scrolling.

Rebase, add some padding at the bottom

Nov 2 2022, 2:17 AM

Oct 25 2022

michal added a comment to D5462: [keyserver] Update userStore from chat creation url.

I think you were added when you requested changes? :)

Oct 25 2022, 8:00 AM
michal updated the diff for D5463: [web] Call searchUsers in chat composer.

Rebase

Oct 25 2022, 7:10 AM
michal updated the diff for D5462: [keyserver] Update userStore from chat creation url.

Revert the removal of useEffect

Oct 25 2022, 7:07 AM
michal added inline comments to D5462: [keyserver] Update userStore from chat creation url.
Oct 25 2022, 7:03 AM
michal updated the diff for D5472: [web] Calendar scrolling.

Used other method, without position: absolute. Moved padding-top to margin in nav because otherwise some of the calendar and scrollbar were cutoff at the bottom.

Oct 25 2022, 6:34 AM
michal added inline comments to D5462: [keyserver] Update userStore from chat creation url.
Oct 25 2022, 2:18 AM
michal updated the summary of D5462: [keyserver] Update userStore from chat creation url.
Oct 25 2022, 2:16 AM
michal added a comment to D5466: [web] Don't display current user in UserList.

We handle this issue correctly on native (here).

Oct 25 2022, 2:08 AM
michal requested review of D5472: [web] Calendar scrolling.
Oct 25 2022, 1:01 AM

Oct 24 2022

michal requested review of D5466: [web] Don't display current user in UserList.
Oct 24 2022, 8:46 AM
michal updated the diff for D5462: [keyserver] Update userStore from chat creation url.

Changed awaits to not use them directly. Now we await in two places:

  • await before fetchKnownUserInfos so the relationships are updated
  • inside navInfoPromise to get validUserIDs

We don't have to await before calendarQuery because it uses only the startDate and endData fields.

Oct 24 2022, 8:24 AM
michal requested review of D5463: [web] Call searchUsers in chat composer.
Oct 24 2022, 6:43 AM
michal requested review of D5462: [keyserver] Update userStore from chat creation url.
Oct 24 2022, 6:27 AM

Oct 20 2022

michal updated the diff for D5343: [web] Use plain Button instead of html elements.

Rebase

Oct 20 2022, 8:02 AM
michal updated the diff for D5342: [web] Introduce plain Button variant.

Rebase

Oct 20 2022, 7:38 AM
michal updated the diff for D5327: [web] Use <a> instead of <div> in some places.

Rebase

Oct 20 2022, 7:20 AM
michal updated the diff for D5324: [web] Use Button for relationship buttons.

Update

Oct 20 2022, 7:15 AM
michal updated the diff for D5323: [web] Use Button in join chat button.

Rebase

Oct 20 2022, 6:58 AM
michal updated the summary of D5322: [web] Use Button for text buttons.
Oct 20 2022, 6:51 AM
michal updated the diff for D5322: [web] Use Button for text buttons.

Fix small issue with RETRY? button.

Oct 20 2022, 6:50 AM