Merge pull request #3806 from AllskyTeam/Days-to-keep-checks

Days to keep checks
AllskyTeam · Aug 12, 2024 · 22a5852 · 22a5852
2 parents a3d22ac + ed9672c
commit 22a5852
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/config_repo/options.json.repo b/config_repo/options.json.repo
@@ -566,6 +566,7 @@
 "label" : "Days To Keep",
 "type" : "integer",
 "carryforward" : true,
+"checkchanges" : true,
 "booldependson" : "savedaytimeimages OR savenighttimeimages"
 },
 {
@@ -1806,6 +1807,7 @@
 "label" : "Days To Keep on Pi Website",
 "type" : "integer",
 "carryforward" : true,
+"checkchanges" : true,
 "booldependson" : "uselocalwebsite"
 },
 {
@@ -1831,6 +1833,7 @@
 "label" : "Days To Keep on Remote Website",
 "type" : "integer",
 "carryforward" : true,
+"checkchanges" : true,
 "booldependson" : "useremotewebsite",
 "display" : false
 },

diff --git a/scripts/makeChanges.sh b/scripts/makeChanges.sh
@@ -778,6 +778,15 @@ do
 			fi
 			;;
 
+		"daystokeep" | "daystokeeplocalwebsite" | "daystokeepremotewebsite")
+			if [[ ${NEW_VALUE} -gt 0 ]]; then
+:	# TODO: Check how many days images there are of the specified type.
+	# For remote website, query the website for the number (to be implemented).
+	# If MORE than NEW_VALUE, warn the user since those images will be deleted
+	# at the next endOfNight.sh run.
+			fi
+			;;
+
 		"uselogin")
 			if [[ ${NEW_VALUE} == "false" ]]; then
 				MSG="${wWARNING}WARNING: "