Skip to content
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

[dashboard] Allow Projects for Individuals #4966

Merged
merged 1 commit into from
Aug 4, 2021
Merged

Conversation

jankeromnes
Copy link
Contributor

@jankeromnes jankeromnes commented Jul 27, 2021

Fixes #4920

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Aug 3, 2021

@AlexTugarev Okay, I believe I've now fixed all the Teams & Project URLs (they all now use either /${team.slug}/... or /projects/... as slug, depending on team- vs user-project).

I've also adjusted the top menu to work with individual projects as well, and I've made sure every page sets a window title (by slightly abusing the Header component).

Please take another look + test again (deployment already has GH app) + let me know what you think! 🙏

@jankeromnes jankeromnes requested a review from AlexTugarev August 3, 2021 11:54
@AlexTugarev
Copy link
Member

AlexTugarev commented Aug 3, 2021

Comments/notes:

  • in general: having Projects under personal account is neat and it feels natural! 👍🏻
  • 🎉 thanks for fixing the window.title issue 💯
  • question: why is re-selecting of the personal account from the top menu navigating to /workspaces and not /projects? this is different with teams. still undecided, I'm not against it, just trying to understand the difference ;-)
  • when opening the New Project wizard from the personal Projects page (New Project button below) it will ask me for the team. This is not the case when starting out from a team's Projects page. This seems to be inconsistent. Let's decide on a single strategy here. Maybe @gtsiolis can assist here?

Screen Shot 2021-08-03 at 14 56 09

* `Rerun Prebuild` seems to be coined to teams. At least it fails with `No team found`

@gtsiolis
Copy link
Contributor

gtsiolis commented Aug 3, 2021

why is re-selecting of the personal account from the top menu navigating to /workspaces and not /projects?

@AlexTugarev I agree that eventually this could probably default to /projects just like with any other team. See also #4966 (comment). It could be fine to also fix this in a smaller and separate PR if needed.

🍊 🍊 🍊 🍊

This is not the case when starting out from a team's Projects page. This seems to be inconsistent. Let's decide on a single strategy here. Maybe @gtsiolis can assist here?

When a user tries to create a project from within a team page it could make sense to automatically associate that project with the team they were looking at, right?

suggestion: When 🅰️ going though the onboarding flow (/new) or 🅱️ having the personal account selected, it could make sense for both cases to promt the user for a team selection including the personal account as a team option as seen in the relevant specs in #4948, see Select Team. Cross-posting below the relevant spec. Feel free to ignore the footnote about Teams & Projects pricing. What do you think? ❓ See also relevant discussion.

Team Selection for New Projects under Personal Account
127672643-cfc64766-0208-427a-8a9d-6c05b400dc74 (1)

@svenefftinge
Copy link
Member

when opening the New Project wizard from the personal Projects page (New Project button below) it will ask me for the team. This is not the case when starting out from a team's Projects page. This seems to be inconsistent. Let's decide on a single strategy here.

I think it is good to ask in one but not the other case, because we want to help users put their projects below teams. So it makes sense to suggest it in that case. When I crate a project from a team there is no need to ask users to choose another team.

@AlexTugarev
Copy link
Member

After double-checking with gitpod-staging.com, it seems prebuilds aren't working here. I cannot see prebuilds running. Server logs do indicate prebuild registration, but there aren't any. @jankeromnes, could you check the webhook URL, please?

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Aug 3, 2021

Many thanks for the reviews!!

After double-checking with gitpod-staging.com, it seems prebuilds aren't working here. I cannot see prebuilds running. Server logs do indicate prebuild registration, but there aren't any. @jankeromnes, could you check the webhook URL, please?

Interesting. With my integration, I was able to trigger a prebuild by:

  1. Installing the app janx-gitpod on https://github.com/jankeromnes/gitpod-staging-prebuilds (while creating a new Project)
  2. Creating a new Pull Request (i.e. branch) on that repository
  3. See the new prebuild appear in the Prebuilds list

Not sure why you didn't see prebuilds. Maybe the janx-gitpod app wasn't installed on the repository you tried with? Or could be a different problem in this PR? (No idea where to look, but I could test again tomorrow -- also I believe the webhook URLs were correct and still are correct.)


Also, what items are still blocking this PR from being merged?

  • The "onboarding flow asks for which team when creating from user" inconsistency was brought up (because when creating from a team, the onboarding flow does not ask & pre-selects the relevant team). However, Sven pointed out that the inconsistency is desirable (asking again when creating from a user encourages people to create teams, which is what we prefer). So, no action required here.

  • The "projects is the default tab for teams, but not for users" inconsistency was brought up. This should likely be fixed in the future (i.e. maybe Projects also the default tab for users), however doing that now is impractical, because of the Teams & Projects UI on/off flag (when off, we don't want a default page that "doesn't exist"). So, maybe we fix this later, when Teams & Projects is enabled generally.

  • Prebuilds didn't work for Alex. Jan can try to test prebuilds again tomorrow.

  • ... anything else still blocking a lgtm? 😇

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Aug 4, 2021

Gitpod deployment, I summon thee! 🧙

/werft run

👍 started the job as gitpod-build-jx-user-projects.28

@svenefftinge
Copy link
Member

svenefftinge commented Aug 4, 2021

anything else still blocking a lgtm? 😇

FWIW, to me it lVerygtm 🎉 when I tested it yesterday and looked over the code, but since you had intense conversations with Alex, I think he should do the approval.

@AlexTugarev
Copy link
Member

  • Rerun Prebuild seems to be coined to teams. At least it fails with No team found

@jankeromnes, this still fails.

@AlexTugarev
Copy link
Member

AlexTugarev commented Aug 4, 2021

Screen Shot 2021-08-04 at 08 55 55

🎉 Prebuilds are shown in personal and team project!

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Aug 4, 2021

Woohoo! Thanks Alex! 🎉

(To be safe, I even recorded a quick screencast to demo prebuilds in action 😁 but I guess it's no longer needed!)

  • Rerun Prebuild seems to be coined to teams. At least it fails with No team found

Oh! I totally overlooked this comment. 🙈 Will check & fix.

@AlexTugarev
Copy link
Member

AlexTugarev commented Aug 4, 2021

/lgtm 👍🏻 👍🏻 👍🏻

/hold @jankeromnes, please have a look at the last two comments of mine.

@roboquat
Copy link
Contributor

roboquat commented Aug 4, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AlexTugarev

Associated issue: #4920

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Aug 4, 2021

Fixed the two nits, and added a dependency on teams for all "project-loading effects": https://github.com/gitpod-io/gitpod/compare/df93235422df3d3646f4e7f565c3d644eaa4d11a..98080ef4db388dfc0acec48e779ba416894f7bf8

Carrying over the /lgtm from #4966 (comment) 😇

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.

[Dashboard] Allow Projects for Individuals
5 participants