Debsmita Santra

Results 53 comments of 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. ![image](https://private-user-images.githubusercontent.com/26255262/357635615-13d9747f-1e39-4416-b6c8-df5e5fee1ad2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjM2MTQ5NjgsIm5iZiI6MTcyMzYxNDY2OCwicGF0aCI6Ii8yNjI1NTI2Mi8zNTc2MzU2MTUtMTNkOTc0N2YtMWUzOS00NDE2LWI2YzgtZGY1ZTVmZWUxYWQyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MTQlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODE0VDA1NTEwOFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTM4NDM4YTM5NGZiZWYzOTU4NzU2ZWVlNDhhOWQxNTYzZjIxZmVhZTUzYzFiZWUyYWM4N2JiM2M0YTI5YmU5OWQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.7u4vqmEJdfzWI9GGWVDu9TQuZBaRwqmCzuyEUMJeJk8) > Fixed ![Screenshot 2024-08-14 at 11 29 23 AM](https://github.com/user-attachments/assets/80e287c5-f24b-417d-8b02-4ad5323489f2) > *...

> Works great 👏 Only a small change request. The Organization URL should be trimmed too. ![image](https://private-user-images.githubusercontent.com/26255262/362063323-0a2d769d-7da8-4eaf-941a-5d65481b3864.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ4MDkzOTcsIm5iZiI6MTcyNDgwOTA5NywicGF0aCI6Ii8yNjI1NTI2Mi8zNjIwNjMzMjMtMGEyZDc2OWQtN2RhOC00ZWFmLTk0MWEtNWQ2NTQ4MWIzODY0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODI4VDAxMzgxN1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY1NzliYzY0Nzc5N2Q3OWYxNmM1OWU0OGUxNDY3ZGMyNmQ0NzRlNWYxN2IxN2EyZjdlODhjZWY3MGE0MjczN2QmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.8vRXM7KZFP4CrEjG0DL-EX50Bd48OzVmMJ-Oh5rAgOo) Fixed ![Screenshot 2024-08-28 at 11 30 43 AM](https://github.com/user-attachments/assets/720ce37d-4af0-42a5-a86c-6bf381488aad)

@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...