Skip to content

Conversation

@konrad147
Copy link
Contributor

@konrad147 konrad147 commented Aug 12, 2025

This pull request improves the behavior and flexibility of the contact point selection component by allowing more control over how and when contact point data is refreshed. The main changes include adding support for passing query options to the contact points fetching hook and updating the selector component to use these options for better data freshness.

Contact point data fetching improvements:

  • Added support for passing query options (such as refetchOnFocus and refetchOnMountOrArgChange) to the useListContactPoints hook, allowing callers to control data refetching behavior.
  • Updated ContactPointSelector to use the new query options, ensuring the contact points list stays up-to-date when the component mounts or the window regains focus.

@konrad147 konrad147 added area/alerting Grafana Alerting area/frontend no-backport Skip backport of PR labels Aug 12, 2025
@github-project-automation github-project-automation bot moved this to In review in Alerting Aug 12, 2025
@konrad147
Copy link
Contributor Author

/deploy-to-hg

@ephemeral-instances-bot
Copy link

  • Preparing your instance. A comment containing your instance's url will be added to this PR when the instance is ready.
  • Your instance will be ready in ~10 minutes. Follow the workflow progress
  • Slack channel: #proj-ephemeral-hg-instances
  • Building instance with alerting/poc-contact-point-channel oss branch and main enterprise branch. How to choose a branch

@ephemeral-instances-bot
Copy link

@konrad147 konrad147 self-assigned this Aug 13, 2025
@konrad147 konrad147 changed the title Alerting: Contact points update events Alerting: Auto refresh contact points in the rule form Aug 13, 2025
@konrad147 konrad147 marked this pull request as ready for review August 13, 2025 13:45
@konrad147 konrad147 requested a review from a team as a code owner August 13, 2025 13:45
@konrad147 konrad147 requested review from gillesdemey, laurenashleigh and soniaAguilarPeiron and removed request for a team August 13, 2025 13:45
@konrad147
Copy link
Contributor Author

/deploy-to-hg

@ephemeral-instances-bot
Copy link

  • Preparing your instance. A comment containing your instance's url will be added to this PR when the instance is ready.
  • Your instance will be ready in ~10 minutes. Follow the workflow progress
  • Slack channel: #proj-ephemeral-hg-instances
  • Building instance with alerting/poc-contact-point-channel oss branch and main enterprise branch. How to choose a branch

@gillesdemey
Copy link
Contributor

nit: prefer the first arg to the hook to be an empty object or something. Maybe later we will add some config option we want to support and I'd like the hook behaviour options to be somewhat separate from that.

@ephemeral-instances-bot
Copy link

@konrad147
Copy link
Contributor Author

/deploy-to-hg

@ephemeral-instances-bot
Copy link

  • Preparing your instance. A comment containing your instance's url will be added to this PR when the instance is ready.
  • Your instance will be ready in ~10 minutes. Follow the workflow progress
  • Slack channel: #proj-ephemeral-hg-instances
  • Building instance with alerting/poc-contact-point-channel oss branch and main enterprise branch. How to choose a branch

@ephemeral-instances-bot
Copy link

Copy link
Contributor

@gillesdemey gillesdemey left a comment

Choose a reason for hiding this comment

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

Stamp of approval!

@gillesdemey gillesdemey merged commit e21eda9 into main Aug 13, 2025
137 of 139 checks passed
@gillesdemey gillesdemey deleted the alerting/poc-contact-point-channel branch August 13, 2025 14:54
@github-project-automation github-project-automation bot moved this from In review to Done in Alerting Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants