Skip to content

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fairlydurable
Copy link
Contributor

  • 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
Copy link
Member

@cretz cretz left a 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
Copy link
Member

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.

Copy link
Contributor Author

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?

Copy link
Member

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.

Copy link
Contributor

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.

Copy link
Contributor

@axfelix axfelix left a 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants