Skip to content

Implement getInAppUrl for alerting_rule_template saved object#234686

Merged
MichelLosier merged 5 commits intoelastic:mainfrom
MichelLosier:add-alerting-rule-template-linking
Sep 19, 2025
Merged

Implement getInAppUrl for alerting_rule_template saved object#234686
MichelLosier merged 5 commits intoelastic:mainfrom
MichelLosier:add-alerting-rule-template-linking

Conversation

@MichelLosier
Copy link
Contributor

@MichelLosier MichelLosier commented Sep 10, 2025

Summary

Resolves: https://github.com/elastic/ingest-dev/issues/5902

  • Adds implementation for getInAppUrl for the alerting_rule_template saved object. This will directly link to the rule creation form with the template values filled in.

Url route support was implemented in: https://github.com/elastic/ingest-dev/issues/5900

Screen.Recording.2025-09-18.at.9.42.18.AM.mov

Manually test

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

@MichelLosier
Copy link
Contributor Author

/ci

@MichelLosier MichelLosier changed the title Impl getInAppUrl for alerting_rule_template SO Implement getInAppUrl for alerting_rule_template saved object Sep 18, 2025
@MichelLosier MichelLosier added backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes labels Sep 18, 2025
@MichelLosier MichelLosier marked this pull request as ready for review September 18, 2025 16:49
@MichelLosier MichelLosier requested review from a team as code owners September 18, 2025 16:49
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #90 / Cloud Security Posture Test adding Cloud Security Posture Integrations CSPM AWS CIS_AWS Single Manual Shared Access CIS_AWS Single Manual Shared Access Workflow

Metrics [docs]

✅ unchanged

History

Copy link
Contributor

@jeramysoucy jeramysoucy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM from platform security perspective - there are no changes to encrypted types

@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Sep 18, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@MichelLosier MichelLosier merged commit cd11190 into elastic:main Sep 19, 2025
19 checks passed
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Sep 24, 2025
…c#234686)

Resolves: elastic/ingest-dev#5902

* Adds implementation for `getInAppUrl` for the `alerting_rule_template`
saved object. This will directly link to the rule creation form with the
template values filled in.

Url route support was implemented in:
elastic/ingest-dev#5900
niros1 pushed a commit that referenced this pull request Sep 30, 2025
Resolves: elastic/ingest-dev#5902

* Adds implementation for `getInAppUrl` for the `alerting_rule_template`
saved object. This will directly link to the rule creation form with the
template values filled in.

Url route support was implemented in:
elastic/ingest-dev#5900
rylnd pushed a commit to rylnd/kibana that referenced this pull request Oct 17, 2025
…c#234686)

Resolves: elastic/ingest-dev#5902

* Adds implementation for `getInAppUrl` for the `alerting_rule_template`
saved object. This will directly link to the rule creation form with the
template values filled in.

Url route support was implemented in:
elastic/ingest-dev#5900
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants