-
Notifications
You must be signed in to change notification settings - Fork 416
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
Add hostname to calendar event (maintenance window) bodies #20235
Add hostname to calendar event (maintenance window) bodies #20235
Conversation
@lucasmrod I'd like to merge #20156 before this one |
@@ -63,7 +63,8 @@ const CalendarEventPreviewModal = ({ | |||
</div> | |||
<div className={`${baseClass}__preview-info__text`}> | |||
{orgName} reserved this time to make some changes to your work | |||
computer. | |||
computer ( | |||
{showGenericPreview ? "ANNAS-DELL" : "Anna's MacBook Pro"}). |
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.
What's this change about?
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.
slightly different copy in the Figma for these two cases. Worth checking, @rachaelshaw was that intended?
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.
@jacobshandling I think we should actually keep it as "Anna's MacBook Pro" for everything. I had changed it in Figma to see how it would look with less pretty display names, but we should keep the more readable one for the preview! Sorry about that, I'll update the Figma.
@lucasmrod thanks for catching this!
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.
Left a few questions!
7c46a7f
to
b77693e
Compare
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.
lgtm!
Addresses #19281
changes/
,