refactor: improve backup usage

• Only one of ynh_backup.system or ynh_backup.apps is required.
• improve readme for the backup.
• If the user put var ynh_backup.system and ynh_backup.apps to Fasle the role must put an error.

added To Do label

changed the description

mentioned in merge request !22 (merged)

assigned to @arthur.boudreault

added Doing label and removed To Do label

changed the description

I think we misunderstood each other on this one. Here is an overview of the script file ynh_backup.sh.j2. You have three choices:

• If you only want to back up YunoHost apps and related data, you "enable" this var: ynh_backup.apps: True
• If you only want to back up YunoHost system and related data (config files...), you "enable" this var: ynh_backup.system: True
• If you want to back up both YunoHost system and apps, you can leave ynh_backup.system and ynh_backup.apps commented or to FALSE because command yunohost backup create does both.

Do I have to modify my script anyway or is it okay?

changed the description

changed the description

changed title from refactor: improve backup usage to refactor: improve backup usage README

created branch 45-refactor-improve-backup-usage to address this issue

mentioned in merge request !39 (merged)

marked the checklist item If the user doesn't provide var ynh_backup.system or ynh_backup.apps the role must put an error. as completed

marked the checklist item improve readme for the backup. as completed

marked the checklist item Only one of ynh_backup.system or ynh_backup.apps is required. as completed

added Need review documentation enhancement labels and removed Doing label

changed title from refactor: improve backup usage README to refactor: improve backup usage

changed the description

marked the checklist item Find a way to prune old backup. as incomplete

marked the checklist item Find a way to prune old backup. as completed

marked the checklist item Find a way to prune old backup. as incomplete

marked the checklist item If the user doesn't provide var ynh_backup.system or ynh_backup.apps the role must put an error. as incomplete

changed the description

mentioned in commit 888d14cd

closed with merge request !39 (merged)

marked the checklist item If the user put var ynh_backup.system and ynh_backup.apps to Fasle the role must put an error. as completed

removed Need review label