Ritesh Patil

Results 23 comments of Ritesh Patil

> @riteshsp2000 Can you add proper tests that you did under 'How should this be tested?' @hariombalhara haven't added the required unit tests. How about I write a unit test...

> You can add a list of manual tests there instead of mentioning just Test A, Test B. There are e2e tests for embed. But there are no tests for...

> @riteshsp2000 Also, I am okay to merge without automation tests. They can be done in a different PR. I just need you to list down the manual testing that...

@PeerRich @alishaz-polymath @hariombalhara I would like to work on this issue. I have gone through the designs and found the [embed skeletons](https://www.figma.com/file/9MOufQNLtdkpnDucmNX10R/1.-Cal-DS?type=design&node-id=1183-43880&t=65GyxlTPcwE25ym0-0) Could you please confirm if these are the...

@hariombalhara @PeerRich I'd like to work on this. We have to update the documentation for embeds right? Currently, the documentation for the Embeds section has 4 sub-sections and must be...

Hey, @PeerRich @alishaz-polymath If this is still up for grabs I'd like to raise a fix.

@Jaibles @shirazdole I find this particular feature promising and would like to work on it. While I do have a few uncertainties regarding the specific implementation particulars, once those are...

@a0m0rajab @PeerRich I would like to work on this if this issue is still up for grabs. though I have a few doubts regarding this, I checked up the email...

Hey @Jaibles @hariombalhara @PeerRich I'd like to work on this issue. Looks like this will be requiring both backend and frontend changes. I can share my solution and we could...