-
Notifications
You must be signed in to change notification settings - Fork 47
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 an alert to point users to the correct relese notes. #245
base: main
Are you sure you want to change the base?
Conversation
@VaishnaviHire @zdtsw ^^ |
src/templates/docs-page.tsx
Outdated
@@ -41,6 +41,11 @@ const DocsPageTemplate = ({ | |||
isWidthLimited | |||
padding={{ default: "padding" }} | |||
> | |||
<Alert variant="info" title="Important Notice" isInline className="pf-u-mb-xl"> | |||
<p> | |||
Please note that the latest information about release notes has been moved <a href="https://github.com/opendatahub-io/opendatahub-operator/releases/latest" target="_blank" rel="noopener noreferrer">here</a>. |
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.
maybe something more like if want to checkout previous v2 releases, go https://github.com/opendatahub-io/opendatahub-operator/releases and use "find release" to search for?
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.
yep I could do that
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.
done
f0854cc
to
f4a640f
Compare
f4a640f
to
c236d41
Compare
i am fine with the change, just needs another eyes on it |
Provide a message to users about the latest release notes.
Description
After some discussion with the team, it seems that the information about a release should have a singe source of truth and that being the https://github.com/opendatahub-io/opendatahub-operator/releases/latest.
So the idea is to provide a useful notice to the users so that they can navigate to the correct one.
How Has This Been Tested?
Merge criteria: