-
Notifications
You must be signed in to change notification settings - Fork 166
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
docs(optimzie): remove telemetry #3778
Conversation
👋 🤖 🤔 Hello! Did you make your changes in all the right places? These files were changed only in optimize/. You might want to duplicate these changes in optimize_versioned_docs/version-3.13.0/.
You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines. |
2ca0e0a
to
c2ac218
Compare
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.
There are still a few references in the diff files, are these to remain as-is?
Lastly, we'll need a redirect. Would you prefer users are routed to https://docs.camunda.io/optimize/self-managed/optimize-deployment/configuration/getting-started/? Let me know if you'd like me to add this 👍
hey @christinaausley! thanks for reaching out! I think it make sense to redirect user to https://docs.camunda.io/optimize/self-managed/optimize-deployment/configuration/getting-started/. I would really appreciate if you added this for me, thank you! |
@misiekhardcore I am assuming we are ready to merge this before release tomorrow? |
@christinaausley yes, we should merge this. Thanks for keeping eye on this 🚀 |
related to #https://github.com/camunda/camunda-optimize/issues/10143
Description
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).