Skip to content
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

Declutter tags #3150

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Declutter tags #3150

wants to merge 14 commits into from

Conversation

napcs
Copy link
Contributor

@napcs napcs commented Oct 11, 2024

This PR reduces tags across the site, removing tags that look like keywords and reducing tags to a small set of tags related to Temporal capabilities.

This does not address keywords themselves, except for a few specific cases.

@napcs napcs requested a review from a team as a code owner October 11, 2024 20:04
- workflow-type
- workflow-visibility
- x509-certificate
- Temporal CLI
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! There were a lot of this page - definitely made the page look a bit cluttered at the bottom.

- sdk
- dotnet-sdk
- developer guide
- .Net SDK
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to have Metrics, Tracing, Logging as tags?

- sdk
- go-sdk
- developer guide
- Activities
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would the tags here be Timeout, Retry, Heartbeat? Or does it stay more general?

- developer guide
- Activities
- .Net SDK
- Temporal SDKs
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is "Temporal SDKs" a useful tag for users?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would certainly get a lot of results, I'd imagine

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am leaning toward dropping it but wasn't sure.

Copy link
Contributor

@fairlydurable fairlydurable left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unblocking this ticket.

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.

5 participants