Kibana: [APM][UI] Broken link in Transaction Metadata page

Created on 23 Mar 2020  路  7Comments  路  Source: elastic/kibana

On the newest stable (7.6.1) If you open up APM, open a service, click on a single Transaction, browse all the way down, and change it to the Metadata tab, there is a link called "How to add labels and other data"(see screenshot below, bottom right corner, by filter).

Expected Behavior
The link should go to the expected metadata page for that version, but it is adding 7.6.1 instead of 7.6 in the URL.
https://www.elastic.co/guide/en/apm/get-started/7.6/metadata.html

Current Behavior
The link to our documentation returns a 404.
https://www.elastic.co/guide/en/apm/get-started/7.6.1/metadata.html

Screenshot
image

NP Migration apm bug v7.7.0

All 7 comments

Pinging @elastic/apm-ui (Team:apm)

This will most likely be fixed when we move to NP. The cause of the problem is that we are using a shimmed value for packageInfo:

https://github.com/elastic/kibana/blob/d8d06e7343d4457d6afa0e6e4b49607abe11f640/x-pack/legacy/plugins/apm/public/new-platform/plugin.tsx#L118-L124

@ogupte were you planning to replace the shims for 7.7 or 7.8?

@sqren we're replacing the shims for 7.8. I'll tag this issue accordingly.

@ogupte were you planning to replace the shims for 7.7 or 7.8?

@sqren Originally targeted 7.7 for this, but will prioritize early in 7.8 development cycle.

Let's make sure that for 7.6.1 and 7.7.0 that all documentation links are not broken. If they are not broken, we'll do this in 7.8, otherwise fix now.

The packageInfo is going to give us major.minor.patch whether or not we're using the new platform.

We should fix the ElasticDocsLink component to just use major/minor.

Going to tag this for 7.7 and open a PR that does that.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

bradvido picture bradvido  路  3Comments

treussart picture treussart  路  3Comments

timroes picture timroes  路  3Comments

socialmineruser1 picture socialmineruser1  路  3Comments

LukeMathWalker picture LukeMathWalker  路  3Comments