Skip to content

Chore: Don't show a "Not found" for public-dashboard fetches if the service is disabled via config #108650

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

Merged
merged 1 commit into from
Jul 25, 2025

Conversation

mmandrus
Copy link
Contributor

@mmandrus mmandrus commented Jul 25, 2025

What is this feature?

A user reported that they disabled public dashboards altogether and started receiving "Not found" toasts on their dashboards. This suppresses the toast if the feature is disabled.

[public_dashboards]
enabled = false # this was causing the error to happen on dashboard loads

@mmandrus mmandrus requested a review from a team as a code owner July 25, 2025 04:12
@mmandrus mmandrus requested review from mdvictor, kaydelaney and a team and removed request for a team July 25, 2025 04:12
@mmandrus mmandrus added this to the 12.2.x milestone Jul 25, 2025
@mmandrus mmandrus enabled auto-merge (squash) July 25, 2025 14:18
@mmandrus mmandrus closed this Jul 25, 2025
auto-merge was automatically disabled July 25, 2025 15:32

Pull request was closed

@mmandrus mmandrus reopened this Jul 25, 2025
@mmandrus mmandrus enabled auto-merge (squash) July 25, 2025 15:33
@mmandrus mmandrus merged commit f69f25b into main Jul 25, 2025
218 of 219 checks passed
@mmandrus mmandrus deleted the mmandrus/suppress-pubdash-404-toastx branch July 25, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants