-
Notifications
You must be signed in to change notification settings - Fork 45
chore(na): add 5.0 release notes #2054
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
|
This document is gonna need some work! We have a lot to highlight in it, creating the PR so others can help fill in the gaps! |
Removed duplicate feature instruction for new deployments.
…date ticket title
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 is starting to look great, @mrjones-plip! 🕔
I went ahead and made a quick review. Here are some extra comments:
- I believe we will need to update the supported Chrome version to
107. - We will need to update the Supported Versions table
- I suggest we add a Hextra banner like the one you tried out here - something like "CHT 5.0 is here! 🎉 Discover [what’s new](some link)"
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
|
Thanks for the review Andra! I think this is close enough to the final form for you to give it a more formal review. Most likely the only big change we'll make is maybe pull admin training cards if that feature doesn't make it because of the bug. |
|
Thank you for the updates @mrjones-plip! I added a few extra suggestions in my earlier comment. Can you look into them, too, before I review? |
|
ah - gotcha - will do! Let me look into those and then request another review when I've updated the docs per your feedback. |
andrablaj
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.
Thanks for clarifying the downtime in the preparation notes. I added some tweaks to make it clear that what's happening is a downtime rather than a bug.
dianabarsan
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 had started the review, Andra requested a huddle, post huddle I did not submit the initial review and some comments are left over.
I think we should really emphasise that the replication and search downtime are temporary. I suggested to use "temporary" maybe as the first part of the phrase so people scanning through the document capture it, but I believe Andra's suggestions are clearer.
Thanks for handling all this, @mrjones-plip !
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Diana Barsan <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
|
Thanks Diana and Andra! |
andrablaj
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.
🚀
Co-authored-by: Andra Blaj <[email protected]>
Co-authored-by: Andra Blaj <[email protected]>
* update all screenshots in Messaging-Overview Page * replace cards with figure shortcode & fix links in messaging overview * ensure figures are wrapped inside <cards> shortcode for mobile-screenshots in Messaging overview
Co-authored-by: Sugat Bajracharya <[email protected]>
dianabarsan
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.
just merge it :D
|
\o/ |
Description
Add 5.0.0 release notes
see medic/cht-core#10000
License
The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.