stefanpejcic / OpenPanel

A highly customizable web hosting control panel built with Docker 🐳
https://openpanel.com
216 stars 33 forks source link

Backups destination edit lacks some crucial details #217

Closed caesarmind closed 1 month ago

caesarmind commented 1 month ago

OpenPanel version(s) affected

0.3.2

What OS are you using?

Ubuntu

Description

When I try edit already added destinations it doesn't let me to change storage limit. Also, when you will add "path field" it should be changable

How to reproduce

...

Possible Solution

No response

Additional Context

No response

stefanpejcic commented 1 month ago

All data from field is validated on save, when editing storage limit for localhost destination it applies it immediately. But for the dummy remote destination example.com it will not accept changes until connection is validated first. This ensures that destination is not saved with incorrect info and in turn stops backups from running. So just edit the info first

"path field" was removed and added to backup jobs-level