Skip to content

Organization-recommended repositories #20559

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Feb 4, 2025
Merged

Organization-recommended repositories #20559

merged 16 commits into from
Feb 4, 2025

Conversation

filiptronicek
Copy link
Member

@filiptronicek filiptronicek commented Jan 28, 2025

Description

Adds onboardingSettings.recommendedRepositories to the org settings API and the corresponding UI to go with it.

Workspace list (active recommendations) Repository settings list Organization onboarding settings
image image image

Related Issue(s)

Fixes CLC-1066

How to test

  1. Visit https://ft-repo-recd38d33fa9.preview.gitpod-dev.com/workspaces
  2. Go import some repositories (doesn't really matter how many, but the UI looks good with 3)
  3. Mark the imported repositories as recommended, as shown on the screenshot in the PR description
  4. Go back to the workspace list, you should see the recommendations populated
  5. Start a workspace on different repository, after starting, the Dashboard should suggest that repository as part of the suggestions

Documentation

This needs docs for sure, see CLC-1103.

https://github.com/gitpod-io/website/pull/5150

What's missing

  • A fixed docs link for the feature
  • Pinning the Suggested repos to the top of the list

Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
@filiptronicek filiptronicek changed the title Organization recommended repositories Organization-recommended repositories Jan 28, 2025
Copy link

socket-security bot commented Jan 28, 2025

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] 🔁 npm/[email protected] None 0 34.7 MB ericfennis

View full report↗︎

Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
Tool: gitpod/catfood.gitpod.cloud
@filiptronicek filiptronicek marked this pull request as ready for review January 30, 2025 10:05
Tool: gitpod/catfood.gitpod.cloud
@geropl
Copy link
Member

geropl commented Jan 30, 2025

I logged in using your link, but don't see any repos on the workspace page:
image

I see the suggested repos, though:
image

🫧 Could that be because I'm not an owner (somehow)? 🤔

@geropl
Copy link
Member

geropl commented Jan 30, 2025

Uh, once I got a workspace in that list it's there!
image
Expected?

@geropl
Copy link
Member

geropl commented Jan 30, 2025

And now suggested is filled with my repos, which feels off.
image

Plus, when opening "Opem sample repository", I just see the "default" examples we ship...?

Not sure what exactly I'd expect, but feels odd atm. Will 🧠 a bit about it...

@filiptronicek
Copy link
Member Author

filiptronicek commented Jan 31, 2025

Thanks for the comments @geropl, seems like I missed a crucial part here, which is to show the section not depending on how many suggestions you have from your recent workspaces, but rather to show them depending on those plus the amount of org-suggested repositories (fixed that in c8989fd (#20559)).

The aim here is to offer a couple of repositories for a new org member to get started, and those suggestions then start filling up with recent recommendations for repos - so the behavior you describe in #20559 (comment) is expected. I'm curious about why this "feels off" to you?

This is deliberate because the section is meant to stick and be there for every user in an installation, and after getting themselves familiarized with Gitpod, it makes sense to fill this section with content relevant to the user.

@geropl
Copy link
Member

geropl commented Feb 4, 2025

@filiptronicek Sorry of I dropped the ball here: Is the preview reflecting the state you mentioned yesterday, and ready to review?

@filiptronicek
Copy link
Member Author

@geropl yes, the preview should reflect the "Open sample repo" change we discussed yesterday.

@geropl
Copy link
Member

geropl commented Feb 4, 2025

I just tested, and here are a couple of things I found:

  • the "Suggested" section feels to noisy somehow, and it's purpose is not clear (enough) to me. Not sure how to solve for this
  • the "Suggested" sections "pops in" and shifts space around, most pronounced if you have 0 "recommended" repos
  • optional 💡 : we could mark the "recommended" repos in the start workspace modal with the same "recommended" bubble we use in the repo list view (there won't be enough horizontal space, so likely a bigger change
  • the "remove from suggested repos" is too hidden, e.g. even when knowing about it I had trouble finding it

@filiptronicek Would you have time to sync?

@filiptronicek
Copy link
Member Author

We synced with @geropl and determined the best approach here is to ship the initial backend implementation (along with the frontend management) and look into improving the UX in a follow-up.

I reverted the workspace list changes in 8fa10f1 (#20559).

@filiptronicek filiptronicek requested a review from geropl February 4, 2025 11:38
Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it. Thank you for the great discussion! ✨ ✔️

Tool: gitpod/catfood.gitpod.cloud
@roboquat roboquat merged commit 754dc9e into main Feb 4, 2025
18 of 19 checks passed
@roboquat roboquat deleted the ft/repo-recommendations branch February 4, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants