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

Unable to clear the hook url from the notification settings #21256

Closed
edmundito opened this issue Jan 11, 2023 — with Slack · 1 comment · Fixed by #21519
Closed

Unable to clear the hook url from the notification settings #21256

edmundito opened this issue Jan 11, 2023 — with Slack · 1 comment · Fixed by #21519
Assignees
Labels
area/frontend Related to the Airbyte webapp team/platform-move type/bug Something isn't working

Comments

Copy link
Contributor

edmundito commented Jan 11, 2023

image

Actual:
When trying to remove the URL from the notification settings and saving, the user gets an error.

Expected:
The user should be able to clear the notification settings when removing the URL.

Steps to reproduce:

  1. Go to notification settings
  2. Add URL and save
  3. Remove URL and save
@krishnaglick
Copy link
Contributor

Likely just need to remove the frontend validation if no value is provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp team/platform-move type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants