Deprecated config default docs link points to C8 docs
Opened this issue · 4 comments
PHWaechtler commented
Context
Currently, the default docs URL in Optimizes ConfigurationValidator points to https://docs.camunda.io/optimize/next/self-managed/optimize-deployment/configuration/system-configuration/
. Example shared in the Hints.
We should consider whether this is still correct now that Optimize 7 docs have been moved. However, its possible that some of the referenced deprecated config locations may not exist in Optimize 7 docs.
I also suspect some of the paths are no longer accurate.
Acceptance Criteria (Required on creation)
- Optimize 7 docs are linked where possible in the source code.
Hints
- See here
- More occurrences https://github.com/search?q=repo%3Acamunda%2Fcamunda-optimize%20%22docs.camunda.io%22&type=code
Links
Breakdown
Dev2QA handover
- Does this ticket need a QA test and the testing goals are not clear from the description? Add a Dev2QA handover comment
yanavasileva commented
mboskamp commented
Did not adjust two occurrences for docs.camunda.io
because they should point to the C8 docs:
- https://github.com/camunda/camunda-optimize/blob/bb3d5ff2073c802d2b7d0449ea2982c3be6f21b8/optimize/util/optimize-commons/src/main/java/org/camunda/optimize/service/util/configuration/condition/CCSaaSCondition.java#L26
- https://github.com/camunda/camunda-optimize/blob/bb3d5ff2073c802d2b7d0449ea2982c3be6f21b8/optimize/util/optimize-commons/src/main/java/org/camunda/optimize/service/util/configuration/condition/CCSMCondition.java#L26
mboskamp commented
CI failures are known flaky tests.
The review of this ticket is low-prio.
PHWaechtler commented