[Dashboard navigation] Inject/extract references

[nickpeihl]

Fixes #154363

Summary

Extracts and injects references to Dashboards in Link panels to allow sharing / importing to other Spaces.

To test this:

1. Create an empty dashboard.
2. Create a Links panel on the dashboard and save to library.
3. Save the dashboard as one or more new dashboards.
4. Update the Links panel with links to the new dashboards.
5. Create two new Spaces, "space1" and "space2" with the default settings.
6. In Saved Objects, click the Actions button for your Links and choose "Copy to Spaces".
7. Copy the Links to "space1".
8. Switch to the "space1" space and verify all the dashboards using that Links panel were included.
9. In Saved Objects, select the Links saved object and export it including all related objects
10. Switch to "space2" and import the Links saved object from the file downloaded in the previous step.
11. Verify all the dashboards were also imported.
12. Inspect the saved object for the Links. The references array should contain objects for each dashboard.
13. The attributes.links[].destination property should be a string in the format of links_<someuuid>_dashboard. This string should match one of the references[].id.

[elasticmachine]

Pinging @elastic/kibana-presentation (Team:Presentation)

[Heenawter]

Tested by-reference links panel (via the steps outlined in the PR description) + tested by-value links panels by importing/copying a dashboard with a by-value link panel. Everything worked as expected, and relationships stayed intact 🎉

Left a comment WRT not having a refName property - personally, I find it a bit confusing, especially when trying to trace the tests? But ultimately I'm good either way 👍

[Heenawter]

Thanks so much for making the refName separate - it made things so much easier to follow IMO 💃 I left one question WRT the external link supported protocols constant, but otherwise looks great!!

[Heenawter]

Should we add mailto to this? We allow mailto for URL drilldowns, and I am using the exact same Regex match for external links so they do as well 🤔

[Heenawter]

This small change makes the tests so much easier to follow - my brain thanks you, hahahah 🤣

[Heenawter]

Approving based on most recent mailto changes 🎉

[kibana-ci]

💔 Build Failed

• Buildkite Build
• Commit: 848daf2
• Interpreting CI Failures

Failed CI Steps

• Jest Integration Tests #1
• Jest Integration Tests #3
• Jest Integration Tests #5
• FTR Configs #12
• FTR Configs #12
• FTR Configs #34
• FTR Configs #34
• FTR Configs #52
• FTR Configs #52
• Post-Build

Test Failures

• [job] [logs] Jest Integration Tests #1 / checking migration metadata changes on all registered SO types detecting migration related changes in registered types
• [job] [logs] FTR Configs #12 / Controls Range Slider Control create and edit edits title and size of an existing control and retains existing range selection
• [job] [logs] FTR Configs #12 / Controls Range Slider Control create and edit edits title and size of an existing control and retains existing range selection
• [job] [logs] Jest Integration Tests #5 / fleet usage telemetry should fetch usage telemetry
• [job] [logs] FTR Configs #52 / Options list control Dashboard options list creation and editing Options List Control creation and editing experience renames an existing control and retains selection
• [job] [logs] FTR Configs #52 / Options list control Dashboard options list creation and editing Options List Control creation and editing experience renames an existing control and retains selection
• [job] [logs] Jest Integration Tests #3 / SO type registrations does not remove types from registrations without updating excludeOnUpgradeQuery
• [job] [logs] FTR Configs #34 / Synthetics API Tests SyncGlobalParams added an integration for previously added monitor
• [job] [logs] FTR Configs #34 / Synthetics API Tests SyncGlobalParams added an integration for previously added monitor

Metrics [docs]

‼️ ERROR: no builds found for mergeBase sha [d51bcd8]

History

• 💔 Build #155188 failed 65f7ba1
• 💔 Build #151975 failed 491ad68

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