Skip to content
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

fix outgoing webhook form race condition which causes integrations to sometimes not be populated #4647

Merged
merged 4 commits into from
Jul 10, 2024

Conversation

joeyorlando
Copy link
Contributor

Which issue(s) this PR closes

Closes https://github.com/grafana/support-escalations/issues/11299

Checklist

  • Unit, integration, and e2e (if applicable) tests updated
  • Documentation added (or pr:no public docs PR label added if not required)
  • Added the relevant release notes label (see labels prefixed w/ release:). These labels dictate how your PR will
    show up in the autogenerated release notes.

@joeyorlando joeyorlando added pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes labels Jul 9, 2024
@joeyorlando joeyorlando requested a review from a team July 9, 2024 21:31
Comment on lines 62 to 64
if (loaderStore.isLoading(ActionKey.FETCH_INTEGRATION_CHANNELS)) {
return null;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

basically this is the main change here, along with wrapping this component in observer

@brojd brojd temporarily deployed to github-pages July 10, 2024 09:34 — with GitHub Actions Inactive
@joeyorlando joeyorlando added this pull request to the merge queue Jul 10, 2024
Merged via the queue into dev with commit d3c1611 Jul 10, 2024
21 checks passed
@joeyorlando joeyorlando deleted the jorlando/fix-outgoing-webhook-form-race-condition branch July 10, 2024 13:27
brojd added a commit that referenced this pull request Sep 18, 2024
… sometimes not be populated (#4647)

# Which issue(s) this PR closes

Closes grafana/support-escalations#11299

## Checklist

- [ ] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] Added the relevant release notes label (see labels prefixed w/
`release:`). These labels dictate how your PR will
    show up in the autogenerated release notes.

---------

Co-authored-by: Dominik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants