Skip to content

Conversation

@bmorelli25
Copy link
Member

This PR fixes a broken link in the RUM Add data tutorial.

The original link used {config.docs.version} to generate a link to the relevant version of the Agent documentation. Because agents are versioned separately from the stack, this resulted in a 404:

Screen Shot 2020-05-19 at 9 56 27 AM

This PR hardcodes the documentation link to current. It's not the ideal solution, but it's better than a 404.

@bmorelli25 bmorelli25 added Team:APM - DEPRECATED Use Team:obs-ux-infra_services. release_note:skip Skip the PR/issue when compiling release notes labels May 19, 2020
@bmorelli25 bmorelli25 requested a review from a team May 19, 2020 16:59
@bmorelli25 bmorelli25 self-assigned this May 19, 2020
@elasticmachine
Copy link
Contributor

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

@bmorelli25 bmorelli25 changed the title docs: update RUM documentation link [APM] Update RUM documentation link in Add data tutorial May 19, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@bmorelli25 bmorelli25 merged commit 4449ab4 into elastic:master May 20, 2020
@bmorelli25 bmorelli25 deleted the hardwire-agent-links-current branch May 20, 2020 02:15
bmorelli25 added a commit to bmorelli25/kibana that referenced this pull request May 20, 2020
bmorelli25 added a commit to bmorelli25/kibana that referenced this pull request May 20, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request May 20, 2020
* master: (33 commits)
  [Saved Objects] adds support for including hidden types in saved objects client (elastic#66879)
  [Discover] Deangularize timechart header (elastic#66532)
  [Discover] Improve and unskip a11y context view test (elastic#66959)
  [SIEM] Refactor Timeline.timelineType draft to Timeline.status draft (elastic#66864)
  docs: update RUM documentation link (elastic#67042)
  [QA] fixup coverage ingestion tests. (elastic#66905)
  [Metrics UI] Add support for multiple groupings to Metrics Explorer (and Alerts) (elastic#66503)
  [Metrics UI] Add sorting for name and value to Inventory View (elastic#66644)
  [Metrics UI] Change Metric Threshold Alert charts to use bar charts (elastic#66672)
  [Uptime] Use React.lazy for alert type registration (elastic#66829)
  [Reporting] Consolidate API Integration Test configs (elastic#66637)
  Allow histogram fields in average and sum aggregations (elastic#66891)
  Fix saved object share link (elastic#66771)
  move role reset into the top level after clause (elastic#66971)
  Automate the labels for any PRs affecting files for the Ingest Management team (elastic#67022)
  [SIEMDPOINT] Move endpoint to siem (elastic#66907)
  server.uuid so is not used (elastic#66963)
  Revert "[ci/stats] fix git metadata collection (elastic#66840)"
  [Uptime] Unmount uptime app properly (elastic#66950)
  [Visualize] Bar chart: Show missing values on chart setting (elastic#66375)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:APM - DEPRECATED Use Team:obs-ux-infra_services. v7.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants