-
Notifications
You must be signed in to change notification settings - Fork 118
(DOCSP-5700): Updated MongoDB Agent banner. #297
Conversation
schmalliso
left a comment
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.
I think the alert could be clearer if rephrased, but from a theme perspective, this is sound.
themes/mms-cloud/page.html
Outdated
| Monitoring, Backup and Automation are merging to become | ||
| the MongoDB Agent. | ||
| <h4 class="alert-heading" style="margin-bottom: 10px;">The MongoDB Agent has been released!</h4> | ||
| Monitoring, Backup and Automation are now the MongoDB Agent. |
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.
This doesn't make sense if you haven't seen the previous alert and fill in that we're referring to the old agents. It might be practical to clarify with something like The Monitoring, Backup, and Automation Agents have been replaced by the MongoDB Agent. Or something.
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.
Agreed. Fixed.
schmalliso
left a comment
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. Lemme know when you want me to merge it!
|
Merged! |
@schmalliso : Just updating the MongoDB Agent banner to be more timely.