mirror of
https://github.com/pelican-dev/panel.git
synced 2026-02-25 11:20:56 +03:00
[Panel] - Proper exception reporting to the end user. #81
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @notAreYouScared on GitHub (May 29, 2024).
Originally assigned to: @lancepioch on GitHub.
Many places in the panel will throw a 500 if a value is required but missing, or it won't report anything at all...
Ex: Eco egg requires a web server port, if left blank the create button appears to do nothing, and nothing tells the end user that the value is required.
@lancepioch commented on GitHub (May 31, 2024):
There's no easy way to do a global resolution because we'd have to hook in between Livewire and Filament. However we can simply utilize vanilla Livewire to hook the exceptions, see my exception method in this PR: https://github.com/pelican-dev/panel/pull/300/files
The idea is to manually specify the exceptions and respond to them appropriately. You can stop the propagation of the normal error window to instead show the Notification message.