Skip to content

Add What's New entry for teams and projects #5126

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

Closed
wants to merge 2 commits into from

Conversation

gtsiolis
Copy link
Contributor

@gtsiolis gtsiolis commented Aug 9, 2021

This will add an entry in the What's New modal for the upcoming Teams & Projects feature.

Fix #5114.

First time adding an entry after the refactoring in #4607. ⚠️

What's New entry for Teams & Projects
Screenshot 2021-08-09 at 11 07 40 PM

@roboquat
Copy link
Contributor

roboquat commented Aug 9, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign geropl after the PR has been reviewed.
You can assign the PR to them by writing /assign @geropl in a comment when ready.

Associated issue: #5114

The full list of commands accepted by this bot can be found 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

@roboquat roboquat requested a review from laushinka August 9, 2021 20:13
@roboquat roboquat added the size/S label Aug 9, 2021
@gtsiolis gtsiolis force-pushed the gt/add-entry-for-teams-and-projects branch from 99eac80 to 070219e Compare August 9, 2021 20:14

import { User } from "@gitpod/gitpod-protocol";
import { WhatsNewEntry } from "./WhatsNew";
import { switchToVSCodeAction } from "./WhatsNew-2021-04";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

thought: I have no idea if this is needed. 😅

</>;
},
newsKey: 'August-2021',
maxUserCreationDate: '2021-08-17',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

thought: Not sure what this does but I've added the target deployment date of next Tuesday (17.08.2021). 🐼

@gtsiolis gtsiolis removed the request for review from laushinka August 9, 2021 20:18
@jankeromnes jankeromnes changed the title Add entry for teams and projects Add What's New entry for teams and projects Sep 1, 2021
Copy link
Contributor

@jankeromnes jankeromnes left a comment

Choose a reason for hiding this comment

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

Many thanks! Added a few thoughts/suggestions in passing 😇

return <>
<div className="border-t border-b border-gray-200 dark:border-gray-800 -mx-6 px-6 pt-6 pb-4">
<p className="pb-2 text-gray-900 dark:text-gray-100 text-base font-medium">Teams & Projects <PillLabel>New Feature</PillLabel></p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">Hey! We're introducing <strong>Teams & Projects</strong> to surface <strong>Prebuilds</strong> in the dashboard. All existing teams will be migrated over the upcoming weeks. No action is required on your side. 🎉</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Thoughts:

  • "Hey!" is quite strong, maybe "Hi there!" is more cheerful and less "shouty"?
  • All teams will be migrated "in the coming weeks" creates the wrong idea that this will happen very fast, and raises too many scary questions about billing etc. Probably safer to say the existing teams remain unchanged for now / might get migrated later "in due time"

Suggestion that you can apply in one click on GitHub:

Suggested change
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">Hey! We're introducing <strong>Teams & Projects</strong> to surface <strong>Prebuilds</strong> in the dashboard. All existing teams will be migrated over the upcoming weeks. No action is required on your side. 🎉</p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">Hi there! We're introducing <strong>Teams & Projects</strong> to surface <strong>Prebuilds</strong> in the dashboard. Existing team plans remain unchanged. No action is required on your side. 🎉</p>

<div className="border-t border-b border-gray-200 dark:border-gray-800 -mx-6 px-6 pt-6 pb-4">
<p className="pb-2 text-gray-900 dark:text-gray-100 text-base font-medium">Teams & Projects <PillLabel>New Feature</PillLabel></p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">Hey! We're introducing <strong>Teams & Projects</strong> to surface <strong>Prebuilds</strong> in the dashboard. All existing teams will be migrated over the upcoming weeks. No action is required on your side. 🎉</p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">You can create as many teams as you need and import repositories as projects from the top navigation. 💡</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Do our users really "need" a feature that hasn't existed until now? Maybe "want" is better.
  • "repositories as projects" sounds a bit too complicated. Maybe just "projects"?
Suggested change
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">You can create as many teams as you need and import repositories as projects from the top navigation. 💡</p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">You can create as many teams as you want and import projects from the top navigation. 💡</p>

<p className="pb-2 text-gray-900 dark:text-gray-100 text-base font-medium">Teams & Projects <PillLabel>New Feature</PillLabel></p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">Hey! We're introducing <strong>Teams & Projects</strong> to surface <strong>Prebuilds</strong> in the dashboard. All existing teams will be migrated over the upcoming weeks. No action is required on your side. 🎉</p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">You can create as many teams as you need and import repositories as projects from the top navigation. 💡</p>
<p className="pb-2 text-gray-500 dark:text-gray-400 text-sm">We welcome any input on this first iteration in <a href="https://github.com/gitpod-io/gitpod/issues/5095" className="learn-more">the feedback issue</a>. 📝</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Please also link to the (upcoming) blog post to learn more, and add a screenshot e.g. like this one on Slack (internal)?

},
newsKey: 'August-2021',
maxUserCreationDate: '2021-08-17',
actionAfterSeen: switchToVSCodeAction,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this (this switches users from Theia to VSCode when they accept the Modal -- this is not relevant here)

@gtsiolis gtsiolis added feature: teams and projects [DEPRECATED] Please, use feature: organizations or feature: projects labels instead. and removed 🧑‍🚀 crew: teams and projects labels Sep 14, 2021
@stale
Copy link

stale bot commented Sep 28, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Sep 28, 2021
@stale stale bot closed this Oct 3, 2021
@gtsiolis gtsiolis deleted the gt/add-entry-for-teams-and-projects branch May 5, 2022 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: dashboard do-not-merge/hold feature: teams and projects [DEPRECATED] Please, use feature: organizations or feature: projects labels instead. meta: stale This issue/PR is stale and will be closed soon size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add what's new modal entry for teams and projects
3 participants