Mirrors/blakeblackshear.frigate
1
0
Fork 0
mirror of https://github.com/blakeblackshear/frigate.git synced 2025-07-26 13:47:03 +02:00
Code Issues Projects Releases Wiki Activity

Fix webUI success / error messages (#7820)

Browse Source 
* Fix export error handling

* Ensure that config editor success / error is updated each time

* Set response

* Formatting
This commit is contained in:
Nicolas Mowen 2023-09-16 03:59:50 -06:00 committed by GitHub
parent 9185753322
commit b9e6afa659
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
frigate/http.py
View File

@ -1646,7 +1646,12 @@ def export_recording(camera_name: str, start_time, end_time):
) )
if recordings_count <= 0: if recordings_count <= 0:
return "No recordings found for time range", 400 return make_response(
jsonify(
{"success": False, "message": "No recordings found for time range"}
),
400,
)
exporter = RecordingExporter( exporter = RecordingExporter(
current_app.frigate_config, current_app.frigate_config,

3
web/src/routes/Config.jsx
View File

@ -28,10 +28,13 @@ export default function Config() {
}) })
.then((response) => { .then((response) => {
if (response.status === 200) { if (response.status === 200) {
setError('');
setSuccess(response.data); setSuccess(response.data);
} }
}) })
.catch((error) => { .catch((error) => {
setSuccess('');
if (error.response) { if (error.response) {
setError(error.response.data.message); setError(error.response.data.message);
} else { } else {

2
web/src/routes/Export.jsx
View File

@ -58,7 +58,7 @@ export default function Export() {
} }
}) })
.catch((error) => { .catch((error) => {
if (error.response) { if (error.response?.data?.message) {
setMessage({ text: `Failed to start export: ${error.response.data.message}`, error: true }); setMessage({ text: `Failed to start export: ${error.response.data.message}`, error: true });
} else { } else {
setMessage({ text: `Failed to start export: ${error.message}`, error: true }); setMessage({ text: `Failed to start export: ${error.message}`, error: true });