You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This makes it not ideal to return a ValidationException in Saving events, nor for other cases where unexpected errors occur since the user must refresh the page to reset the state.
Steps to Reproduce
Go to a settings page of an extension
Click the save button (make sure an error response will be returned from the saving endpoint)
Expected Behavior
Save button is not in loading state after receiving the server response.
Current Behavior
While saving settings, some errors might occur, however, the state of the save button won't reset if an error occurs in
saveSettings()
framework/framework/core/js/src/admin/components/AdminPage.tsx
Line 412 in 721e2ea
This makes it not ideal to return a
ValidationException
inSaving
events, nor for other cases where unexpected errors occur since the user must refresh the page to reset the state.Steps to Reproduce
Expected Behavior
Save button is not in loading state after receiving the server response.
Screenshots
Environment
Output of
php flarum info
Possible Solution
Add a catch handler for
saveSettings()
's promise and reset the save button stateAdditional Context
No response
The text was updated successfully, but these errors were encountered: