Page MenuHomePhabricator
Feed Advanced Search

May 10 2023

ginsu requested review of D7781: [landing] animate mobile nav component.
May 10 2023, 7:04 AM
ginsu added a comment to D7766: [landing] memoize header and footer.

This has literally no effect

May 10 2023, 3:59 AM

May 9 2023

ginsu added inline comments to D7764: [web] Basic `CommunityCreationModal` layout/styling.
May 9 2023, 8:49 AM
ginsu requested review of D7766: [landing] memoize header and footer.
May 9 2023, 8:47 AM
ginsu added a comment to D7764: [web] Basic `CommunityCreationModal` layout/styling.

could you link the figma?

May 9 2023, 8:26 AM
ginsu requested review of D7763: [landing] introduce mobile nav compoment.
May 9 2023, 7:09 AM
ginsu requested review of D7759: [landing] update landing background color.
May 9 2023, 6:42 AM
ginsu requested review of D7757: [landing] remove star background.
May 9 2023, 6:16 AM
ginsu requested review of D7756: [landing] introduce colors from design system.
May 9 2023, 5:58 AM
ginsu requested review of D7755: [landing] introduce new typography design system.
May 9 2023, 5:47 AM
ginsu accepted D7745: [lib] Fix circular dependency.
May 9 2023, 3:01 AM
ginsu added a comment to D7745: [lib] Fix circular dependency.

lgtm thanks for fixing this!

May 9 2023, 3:01 AM

May 8 2023

ginsu closed D7737: [lib/native] add more icons to comm icon pack.
May 8 2023, 11:59 AM
ginsu committed rCOMMe32fedf8b34d: [lib/native] add more icons to comm icon pack (authored by ginsu).
[lib/native] add more icons to comm icon pack
May 8 2023, 11:59 AM
ginsu requested review of D7737: [lib/native] add more icons to comm icon pack.

Discussed this with @ashoat and Ted, but we are going to land this for now to unblock @rohan and the task above is a follow up task to make all our comm icons uniform at 16x16px

May 8 2023, 11:57 AM
ginsu requested changes to D7745: [lib] Fix circular dependency.

The code looks good, but I actually pitched this idea to @ashoat initially and we talked about finding another solution. Thinking out lout right now, but wouldn't it be possible to decouple this cycle by moving defaultNotificationPayloadValidator out of account-types and possibly into validation-utils? I think this would also be better since there are no other validators in account-utils and account-utils should exclusively just be for types

May 8 2023, 10:56 AM
ginsu planned changes to D7737: [lib/native] add more icons to comm icon pack.

Does the sizing look right? Can't really tell from the images from the Test Plan, but as long as things looked right during your testing.

I actually just double checked using the inspector tool, and noticed that these new icons are actually a tad bit smaller, will have ted fix this and send me some new icons

May 8 2023, 9:39 AM

May 6 2023

ginsu requested review of D7737: [lib/native] add more icons to comm icon pack.
May 6 2023, 3:17 PM

May 2 2023

ginsu accepted D7703: [native] Get rid of null check in `useUploadSelectedMedia`.
May 2 2023, 12:23 PM

May 1 2023

ginsu accepted D7698: [native] Fix ComposedMessage width constraint to only include content.

patched in the changes and played around with the views and everything looks good, no more bounds being overflown. Thanks for fixing this

May 1 2023, 1:48 PM
ginsu accepted D7696: [native] Clean up ComposedMessage.
May 1 2023, 1:33 PM
ginsu accepted D7697: [native] Rename messageBoxStyle to contentStyle in SwipeableMessage.
May 1 2023, 1:33 PM
ginsu closed D7692: [native] change useUploadSelectedMedia callback return type from void to mixed.
May 1 2023, 10:16 AM
ginsu committed rCOMM0cdcd32df3be: [native] change useUploadSelectedMedia callback return type from void to mixed (authored by ginsu).
[native] change useUploadSelectedMedia callback return type from void to mixed
May 1 2023, 10:16 AM
ginsu requested review of D7692: [native] change useUploadSelectedMedia callback return type from void to mixed.
May 1 2023, 7:29 AM
ginsu added inline comments to D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
May 1 2023, 7:13 AM

Apr 30 2023

ginsu accepted D7686: [native] Add disabled state to RegistrationButton.
Apr 30 2023, 10:19 PM
ginsu closed D7642: [native] improve user experience for remvoing an avatar.
Apr 30 2023, 10:17 PM
ginsu closed D7641: [native] hardcode avatar feature flag to true.
Apr 30 2023, 10:17 PM
ginsu closed D7646: [native] introduce thread avatar camera modal.
Apr 30 2023, 10:17 PM
ginsu committed rCOMM083b8d82f0a1: [native] improve user experience for remvoing an avatar (authored by ginsu).
[native] improve user experience for remvoing an avatar
Apr 30 2023, 10:17 PM
ginsu committed rCOMM3bb311aac241: [native] hardcode avatar feature flag to true (authored by ginsu).
[native] hardcode avatar feature flag to true
Apr 30 2023, 10:17 PM
ginsu committed rCOMM6f746f5b1126: [native] introduce thread avatar camera modal (authored by ginsu).
[native] introduce thread avatar camera modal
Apr 30 2023, 10:17 PM
ginsu closed D7645: [native] improve factoring out redundant logic for saving an image thread avatar.
Apr 30 2023, 10:17 PM
ginsu closed D7644: [native] move navigate to emoji thread avatar creation function to be within edit thread avatar.
Apr 30 2023, 10:17 PM
ginsu committed rCOMM6309c77470c9: [native] improve factoring out redundant logic for saving an image thread avatar (authored by ginsu).
[native] improve factoring out redundant logic for saving an image thread avatar
Apr 30 2023, 10:17 PM
ginsu closed D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
Apr 30 2023, 10:17 PM
ginsu committed rCOMMf65bfddfcd95: [native] move navigate to emoji thread avatar creation function to be within… (authored by ginsu).
[native] move navigate to emoji thread avatar creation function to be within…
Apr 30 2023, 10:17 PM
ginsu committed rCOMMc29df39551af: [native] have upload thread avatar processing/uploading loader only appear for… (authored by ginsu).
[native] have upload thread avatar processing/uploading loader only appear for…
Apr 30 2023, 10:17 PM
ginsu closed D7684: [native] move selectFromGalleryAndUpdateThreadAvatar to edit thread avatar provider.
Apr 30 2023, 10:17 PM
ginsu committed rCOMMbb875e7f83b4: [native] move selectFromGalleryAndUpdateThreadAvatar to edit thread avatar… (authored by ginsu).
[native] move selectFromGalleryAndUpdateThreadAvatar to edit thread avatar…
Apr 30 2023, 10:17 PM
ginsu updated the diff for D7642: [native] improve user experience for remvoing an avatar.

rebase before landing

Apr 30 2023, 9:54 PM
ginsu updated the diff for D7641: [native] hardcode avatar feature flag to true.

rebase before landing

Apr 30 2023, 9:53 PM
ginsu updated the diff for D7646: [native] introduce thread avatar camera modal.

rebase before landing

Apr 30 2023, 9:53 PM
ginsu updated the diff for D7645: [native] improve factoring out redundant logic for saving an image thread avatar.

rebase before landing

Apr 30 2023, 9:50 PM
ginsu updated the diff for D7644: [native] move navigate to emoji thread avatar creation function to be within edit thread avatar.

rebase before landing

Apr 30 2023, 9:49 PM
ginsu updated the diff for D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.

rebase before landing

Apr 30 2023, 9:48 PM
ginsu updated the diff for D7684: [native] move selectFromGalleryAndUpdateThreadAvatar to edit thread avatar provider.

rebase before landing

Apr 30 2023, 9:46 PM
ginsu added inline comments to D7646: [native] introduce thread avatar camera modal.
Apr 30 2023, 10:01 AM
ginsu updated the diff for D7646: [native] introduce thread avatar camera modal.

address comments

Apr 30 2023, 10:00 AM
ginsu updated the diff for D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.

address comments

Apr 30 2023, 9:55 AM
ginsu added inline comments to D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
Apr 30 2023, 9:55 AM

Apr 28 2023

ginsu updated the summary of D7645: [native] improve factoring out redundant logic for saving an image thread avatar.
Apr 28 2023, 1:18 PM
ginsu updated the diff for D7645: [native] improve factoring out redundant logic for saving an image thread avatar.

rebase

Apr 28 2023, 1:13 PM
ginsu updated the diff for D7644: [native] move navigate to emoji thread avatar creation function to be within edit thread avatar.

rebase

Apr 28 2023, 1:09 PM
ginsu updated the summary of D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
Apr 28 2023, 1:06 PM
ginsu retitled D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated from [native] move selecting from gallery logic from avatar hooks to edit thread avatar to [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
Apr 28 2023, 1:05 PM
ginsu requested review of D7684: [native] move selectFromGalleryAndUpdateThreadAvatar to edit thread avatar provider.
Apr 28 2023, 12:51 PM
ginsu updated the diff for D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.

split diff into two (this is part 2 and part 1 is handeled by D7684)

Apr 28 2023, 12:49 PM
ginsu closed D7679: [native] have saving thread avatar loaders only appear for thread being updated.
Apr 28 2023, 12:44 PM
ginsu committed rCOMMd3224971ec9e: [native] have saving thread avatar loaders only appear for thread being updated (authored by ginsu).
[native] have saving thread avatar loaders only appear for thread being updated
Apr 28 2023, 12:44 PM
ginsu updated the diff for D7679: [native] have saving thread avatar loaders only appear for thread being updated.

one more rebase before landing

Apr 28 2023, 12:42 PM
ginsu closed D7629: [native] introduce edit thread avatar provider.
Apr 28 2023, 11:09 AM
ginsu committed rCOMMfc5e55e7e858: [native] introduce edit thread avatar provider (authored by ginsu).
[native] introduce edit thread avatar provider
Apr 28 2023, 11:09 AM
ginsu updated the diff for D7629: [native] introduce edit thread avatar provider.

one more rebase

Apr 28 2023, 11:08 AM
ginsu updated the diff for D7679: [native] have saving thread avatar loaders only appear for thread being updated.

rebase before landing

Apr 28 2023, 11:04 AM
ginsu updated the diff for D7629: [native] introduce edit thread avatar provider.

rebase before landing

Apr 28 2023, 11:03 AM

Apr 27 2023

ginsu requested review of D7679: [native] have saving thread avatar loaders only appear for thread being updated.
Apr 27 2023, 4:04 PM
ginsu updated the test plan for D7629: [native] introduce edit thread avatar provider.
Apr 27 2023, 3:51 PM
ginsu updated the summary of D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.
Apr 27 2023, 3:51 PM
ginsu updated the summary of D7629: [native] introduce edit thread avatar provider.
Apr 27 2023, 3:32 PM
ginsu updated the diff for D7629: [native] introduce edit thread avatar provider.

simplify callback

Apr 27 2023, 3:30 PM
ginsu added inline comments to D7629: [native] introduce edit thread avatar provider.
Apr 27 2023, 3:24 PM
ginsu added inline comments to D7629: [native] introduce edit thread avatar provider.
Apr 27 2023, 2:59 PM
ginsu updated the diff for D7629: [native] introduce edit thread avatar provider.

split diff (this is the first part)

Apr 27 2023, 2:55 PM
ginsu closed D7624: [native] introduce user avatar camera modal.
Apr 27 2023, 1:48 PM
ginsu committed rCOMM6cb4138efeda: [native] introduce user avatar camera modal (authored by ginsu).
[native] introduce user avatar camera modal
Apr 27 2023, 1:48 PM
ginsu closed D7676: [native] make handlePhotoCapture return mixed instead of void.
Apr 27 2023, 1:43 PM
ginsu committed rCOMM87e0e3e2ddd4: [native] make handlePhotoCapture return mixed instead of void (authored by ginsu).
[native] make handlePhotoCapture return mixed instead of void
Apr 27 2023, 1:43 PM
ginsu closed D7628: [native] improve factoring out redundant logic for saving an image user avatar.
Apr 27 2023, 1:37 PM
ginsu committed rCOMMc8ae0af5dff1: [native] improve factoring out redundant logic for saving an image user avatar (authored by ginsu).
[native] improve factoring out redundant logic for saving an image user avatar
Apr 27 2023, 1:37 PM
ginsu accepted D7638: [native] Return without error if avatar image `selection` is empty.
Apr 27 2023, 1:36 PM
ginsu added inline comments to D7624: [native] introduce user avatar camera modal.
Apr 27 2023, 1:29 PM
ginsu updated the diff for D7624: [native] introduce user avatar camera modal.

address comments

Apr 27 2023, 1:27 PM
ginsu published D7676: [native] make handlePhotoCapture return mixed instead of void for review.
Apr 27 2023, 1:26 PM
ginsu updated the summary of D7624: [native] introduce user avatar camera modal.
Apr 27 2023, 1:25 PM
ginsu added a comment to D7628: [native] improve factoring out redundant logic for saving an image user avatar.

It would be good to explain the reason for the changes here. Are these changes being made because you're going to reuse the factored-out code in a later diff in the stack, so it makes sense to factor out the shared logic here?

Apr 27 2023, 12:45 PM
ginsu updated the diff for D7628: [native] improve factoring out redundant logic for saving an image user avatar.

address comments/rebase before landing

Apr 27 2023, 12:42 PM
ginsu closed D7640: [web] turn off dummy avatar feature flag.
Apr 27 2023, 12:34 PM
ginsu committed rCOMM7816aae2fb74: [web] turn off dummy avatar feature flag (authored by ginsu).
[web] turn off dummy avatar feature flag
Apr 27 2023, 12:34 PM
ginsu closed D7639: [web] fix image avatar image being stretched.
Apr 27 2023, 12:33 PM
ginsu committed rCOMMc045a051665e: [web] fix image avatar image being stretched (authored by ginsu).
[web] fix image avatar image being stretched
Apr 27 2023, 12:33 PM
ginsu updated the diff for D7640: [web] turn off dummy avatar feature flag.

rebase before landing

Apr 27 2023, 11:20 AM
ginsu updated the diff for D7639: [web] fix image avatar image being stretched.

rebase before landing

Apr 27 2023, 11:18 AM
ginsu closed D7622: [native] move navigate to emoji user avatar creation function to be within edit user avatar.
Apr 27 2023, 11:13 AM
ginsu committed rCOMMa73447d07684: [native] move navigate to emoji user avatar creation function to be within edit… (authored by ginsu).
[native] move navigate to emoji user avatar creation function to be within edit…
Apr 27 2023, 11:13 AM
ginsu closed D7621: [native] move selecting from gallery logic from avatar hooks to edit user avatar provider.
Apr 27 2023, 11:11 AM
ginsu committed rCOMMa7a94f7b5f57: [native] move selecting from gallery logic from avatar hooks to edit user… (authored by ginsu).
[native] move selecting from gallery logic from avatar hooks to edit user…
Apr 27 2023, 11:11 AM
ginsu updated the diff for D7622: [native] move navigate to emoji user avatar creation function to be within edit user avatar.

rebase before landing

Apr 27 2023, 11:09 AM
ginsu updated the diff for D7621: [native] move selecting from gallery logic from avatar hooks to edit user avatar provider.

rebase before landing

Apr 27 2023, 10:30 AM