-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
99eac80
to
070219e
Compare
|
||
import { User } from "@gitpod/gitpod-protocol"; | ||
import { WhatsNewEntry } from "./WhatsNew"; | ||
import { switchToVSCodeAction } from "./WhatsNew-2021-04"; |
There was a problem hiding this comment.
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', |
There was a problem hiding this comment.
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). 🐼
There was a problem hiding this 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> |
There was a problem hiding this comment.
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:
<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> |
There was a problem hiding this comment.
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"?
<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> |
There was a problem hiding this comment.
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, |
There was a problem hiding this comment.
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)
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. |
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.⚠️