-
Notifications
You must be signed in to change notification settings - Fork 250
EDU-4412 Clean up extraneous SUPPORT, STABILITY, and DEPENDENCY INFO boxes #3558
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
base: main
Are you sure you want to change the base?
Conversation
fairlydurable
commented
Apr 29, 2025
- Remove general release
- Keep pre-release but caution against coverage
- Keep public preview
- Retain all actual caution and warning content
- Remove general release - Keep pre-release but caution against coverage - Keep public preview - Retain all actual caution and warning content - Excludes all Nexus content
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, though would confirm with server on the visibility stuff
@@ -18,14 +18,6 @@ tags: | |||
|
|||
This page provides an overview of Temporal Visibility. | |||
|
|||
:::tip Support, stability, and dependency info |
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 you should get server verification that they are ok removing this. My opinions concerning this box was more about SDK-side things.
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.
Who would be a good person to connect with?
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 @rodrigozhou or @jmbarzee (or can engage the server team on their internal Slack channel). I just want to confirm they are ok with us removing visibility server version notes.
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.
The Server release notes also contain all these info. So, I'm ok either way if we want to have this in the docs page or not.
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 based on conversation -- let's wait to hear back about Versioning.