Page MenuHomePhabricator

manan (Manan Manchanda)
DisabledEmail Not Verified

Projects

User does not belong to any projects.

User Details

User Since
Sep 30 2022, 6:32 PM (84 w, 6 d)
Roles
Disabled

Recent Activity

Nov 10 2022

manan closed D5588: added glow effect behind modals.
Nov 10 2022, 8:44 AM

Nov 9 2022

manan added a comment to D5588: added glow effect behind modals.

Here's what the glow looks like now

Nov 9 2022, 10:11 PM
manan updated the diff for D5588: added glow effect behind modals.

update

Nov 9 2022, 10:10 PM
manan updated the diff for D5588: added glow effect behind modals.

attempting to remove changes from un-rebased pull

Nov 9 2022, 10:06 PM
manan updated the diff for D5588: added glow effect behind modals.

copied what is done in native/chat/settings/color-selector-modal.react.js

Nov 9 2022, 10:02 PM
manan requested review of D5588: added glow effect behind modals.
Nov 9 2022, 2:00 PM

Nov 2 2022

manan closed D5516: added no-var rule to root level eslint.

Landed

Nov 2 2022, 3:54 PM
manan added inline comments to D5516: added no-var rule to root level eslint.
Nov 2 2022, 11:06 AM
manan updated the diff for D5516: added no-var rule to root level eslint.

rule changed from number to string for readability

Nov 2 2022, 11:03 AM

Nov 1 2022

manan requested review of D5516: added no-var rule to root level eslint.
Nov 1 2022, 11:33 AM

Oct 30 2022

manan added a comment to D5483: added tests to check spoiler regex extracted values.

Landed changes.

Oct 30 2022, 10:36 PM

Oct 27 2022

manan updated the diff for D5483: added tests to check spoiler regex extracted values.
  1. Updating D5483: added tests to check spoiler regex extracted values #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #
  4. If you intended to create a new revision, use:
  5. $ arc diff --create
Oct 27 2022, 8:42 AM

Oct 25 2022

manan requested review of D5483: added tests to check spoiler regex extracted values.
Oct 25 2022, 7:22 PM

Oct 20 2022

manan added a comment to D5358: Added tests for spoiler regex.

So I was basically wondering if we can test whether the RegEx matches what we are actually looking for. So for example, if we are doing something like expect(||hello||).toMatch(spoilerRegex), is there a way to test whether the RegEx is extracting out the "spoiler text", in this case hello?

Oct 20 2022, 1:48 PM
manan abandoned D5447: more descriptive it statements, moved the RegEx text to a comment.

Amended changes to the original diff.

Oct 20 2022, 1:46 PM
manan updated the diff for D5358: Added tests for spoiler regex.

Addressed Rohan's feedback

Oct 20 2022, 1:44 PM
manan requested review of D5447: more descriptive it statements, moved the RegEx text to a comment.
Oct 20 2022, 12:26 PM
manan added a comment to D5358: Added tests for spoiler regex.

Hi @rohan, thank you for your review. To follow up on your points:
1: I've changed the it statements to be more descriptive.
2/3: I'm a bit confused on what you mean by "to actually test the match so we can see if we are matching what is expected" - could you please clarify?
4: I was getting a eslint/prettier problem when I tried to include the actual RegEx in the it statement, so I moved it to a comment. Is this okay?

Oct 20 2022, 12:20 PM

Oct 12 2022

manan requested review of D5358: Added tests for spoiler regex.
Oct 12 2022, 7:32 PM

Oct 1 2022

ashoat renamed manan from mananm12 to manan.
Oct 1 2022, 3:46 AM