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

The Notification Webhook is broken for a single Run State #12468

Closed
5 of 8 tasks
kingabzpro opened this issue Mar 28, 2024 · 2 comments
Closed
5 of 8 tasks

The Notification Webhook is broken for a single Run State #12468

kingabzpro opened this issue Mar 28, 2024 · 2 comments
Assignees
Labels
bug Something isn't working ui Related to the Prefect web interface

Comments

@kingabzpro
Copy link

First check

  • I added a descriptive title to this issue.
  • I used the GitHub search to find a similar issue and didn't find it.
  • I refreshed the page and this issue still occurred.
  • I checked if this issue was specific to the browser I was using by testing with a different browser.

Bug summary

When setting a notification on Prefect OSS Server, only "All run states" state can be used. Attempting to use any other state will result in an error "Error updating notification".

Reproduction

When selecting Discord or Slack webhook option in notification, selecting any run states except "All run states" and pressing Save button results in an error: "Error updating notification".

Error

Here are my Notifications Settings

Screenshot 2024-03-28 223525

Here is the error

image

Here is the error on Slack Webhook

image

Browsers

  • Chrome
  • Firefox
  • Safari
  • Edge

Prefect version

Version:             2.16.3
API version:         0.8.4
Python version:      3.9.18
Git commit:          e3f02c00
Built:               Thu, Mar 7, 2024 4:56 PM
OS/Arch:             win32/AMD64
Profile:             default
Server type:         cloud

Additional context

No response

@kingabzpro kingabzpro added bug Something isn't working needs:triage Needs feedback from the Prefect product team ui Related to the Prefect web interface labels Mar 28, 2024
@aaazzam aaazzam removed the needs:triage Needs feedback from the Prefect product team label Mar 28, 2024
@zhen0
Copy link
Member

zhen0 commented Apr 2, 2024

This has already been resolved by PrefectHQ/prefect-ui-library#2216 and is fixed in the latest version of Prefect.

@zhen0 zhen0 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 2, 2024
@kingabzpro
Copy link
Author

@zhen0 thank you. It is working for me.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Related to the Prefect web interface
Projects
None yet
Development

No branches or pull requests

4 participants