middleware icon indicating copy to clipboard operation
middleware copied to clipboard

Add repos by orgs. Select org first?

Open dhruvagarwal opened this issue 1 year ago • 7 comments

Description:

A user might be a part of multiple orgs and while selecting the repositories, it becomes confusing to find the right org.

Expected behavior:

It would be great to choose the orgs first. This will also help to reduce the clutter in the Add Repos option.

Actual behavior:

Today, it loads up a list of repos across my personal and from other orgs. Screenshot 2024-04-24 at 7 12 13 PM

dhruvagarwal avatar Apr 24 '24 13:04 dhruvagarwal

does #445 solve this?

e-for-eshaan avatar Jun 21 '24 16:06 e-for-eshaan

I don't think so. Adding repos by org might require user to repeatedly add org name for search.

dhruvagarwal avatar Jun 21 '24 17:06 dhruvagarwal

noted!

e-for-eshaan avatar Jun 22 '24 17:06 e-for-eshaan

Hey, I'd be happy to take this up.

If it's possible to strictly enforce the flow of choosing the org before the repo, I think we can change the useReposSearch hook in useTeamsConfig.tsx to only search for orgs, and using that value, render a similar searchbox on the side with the repositories owned by that org (perhaps rename things to useOrgsSearch and useOrgReposSearch?)

This will obviously not let the users search repos directly though. Maybe keep the current box as it is, and alongside it, provide this flow as well? Let me know which implementation you'd prefer.

dieselftw avatar Aug 10 '24 11:08 dieselftw

Hey @hardikbhatia777, we're picking this up internally actually. Apologies this was unassigned.

However, as a good first issue, you could pick up this one if you're interested.

jayantbh avatar Aug 13 '24 10:08 jayantbh

@jayantbh Oh alright. The issue you linked is already solved though. No worries, I'll pick a different one.

dieselftw avatar Aug 13 '24 15:08 dieselftw

Ah I see. Tbh it was picked but not really solved. The associated pr may be closed without merging so you're free to give it a shot anyway.

jayantbh avatar Aug 13 '24 16:08 jayantbh