PALASH BADERIA

Results 102 comments of PALASH BADERIA

> @palashb01 LGTM, you should squash the last two commits. Sure

Hey @amanagr , I have squashed all the commits into one.

Hey @alya @amanagr @N-Shar-ma , I have added a complete node test coverage for the file `popover_menus_data.js` , could you please review it once and suggest if any changes are...

Hey @amanagr , I have made the suggested changes however the node test is failing but locally node tests are passing, Could you please review the PR and suggest what...

> @palashb01 I can take a look once the node tests are passing. All the node tests passed locally on my system, but I don't know why the workflow is...

Hey @amanagr ,I have pushed the final commit with all the node tests passing. Could you please review the PR, Thanks.

Hey @timabbott , I have made all the changes suggested by you, could you please review the PR and suggest if any changes are required, Thanks.

@timabbott just bumping this up, since it's been some time. :)

Hey @alya @sahil839 @amanagr , Could you please review the PR and suggest if any changes are required, Thanks.

> > Overlapping message_content with icons: > > Hm, when do you get overlap like that? I don't think we want buttons and content to overlap. I have attached the...