Debsmita Santra
Debsmita Santra
@ShiranHi Thank you for reviewing this PR. Please find my replies below the questions > Hey @debsmita1 looks good to me. Some questions: > > 1. For the "Selected repositories"...
> > > 2. I think that we can remove the "Repositories" tab appears below the header, we have only one bulk import type at this moment. What do you...
> Thanks @debsmita1, in dev mode I see one issue > > Click on status/refresh breaks on dev mode > > Fixed it! PTAL
> I first take a look from the user perspective before I will review the code as well. > > Just some small ideas/feedback, that we can also fix in...
> Hi @debsmita1 , it works very good now. Awesome job 👏 Just two concerns. > > * For users with permission of an added repo, will the "Edit" action...
> * `Waiting for PR Approval` status doesn't provide a link to open that PR. /cc @ShiranHi > > * On the import "Added repositories" page it shows an action...
> Looks good to me. @debsmita1 Can you confirm that bulk import frontend plugin will be still working after changing endpoint url? @AndrienkoAleksandr The frontend with the API integration hasn't...
> Hi @debsmita1 , thanks for the PR 👏 Looks like bottom button margin is a bit off.  > Fixed  > *...
> Works great 👏 Only a small change request. The Organization URL should be trimmed too.  Fixed 
@ShiranHi I just noticed that the topology icon that I have used here, the same has been used for the navigation icon for clusters on the navigation sidebar in your...