-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Discover] mailto links in data view urls #200070
Conversation
3d84303
to
93c1844
Compare
💚 Build Succeeded
Metrics [docs]Page load bundle
|
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
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.
Looks good to me, great to see corresponding test included! 🎉
@ghudgins you just want this for 9.0? |
ah, didn't realize skipping backporting would only do 9.0. yes, would be nice to have in 8.17 if possible. can wait otherwise |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
Adds `mailto:` as an allowed URL Url formatters in data views. (cherry picked from commit d7a913f)
# Backport This will backport the following commits from `main` to `8.x`: - [[Discover] mailto links in data view urls (#200070)](#200070) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Graham Hudgins","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-14T13:30:10Z","message":"[Discover] mailto links in data view urls (#200070)\n\nAdds `mailto:` as an allowed URL Url formatters in data views.","sha":"d7a913f183cfe35efe7f42bb2d46990abc07296d","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:skip","v9.0.0","Team:DataDiscovery"],"number":200070,"url":"https://github.com/elastic/kibana/pull/200070","mergeCommit":{"message":"[Discover] mailto links in data view urls (#200070)\n\nAdds `mailto:` as an allowed URL Url formatters in data views.","sha":"d7a913f183cfe35efe7f42bb2d46990abc07296d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200070","number":200070,"mergeCommit":{"message":"[Discover] mailto links in data view urls (#200070)\n\nAdds `mailto:` as an allowed URL Url formatters in data views.","sha":"d7a913f183cfe35efe7f42bb2d46990abc07296d"}}]}] BACKPORT--> Co-authored-by: Matthias Wilhelm <[email protected]>
Adds `mailto:` as an allowed URL Url formatters in data views.
Adds `mailto:` as an allowed URL Url formatters in data views.
Adds `mailto:` as an allowed URL Url formatters in data views.
Summary
Adds
mailto:
as an allowed URL Url formatters in data views.Resolves papercut: #179773
Example:
Checklist
Delete any items that are not applicable to this PR.
Risk Matrix
Delete this section if it is not applicable to this PR.
Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.
When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:
For maintainers