WEB_DAYS_TO_KEEP #1076
Merged
WEB_DAYS_TO_KEEP #1076
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
EricClaeys
commented
Mar 15, 2022
EricClaeys
commented
- Similar to NIGHTS_TO_KEEP, but for web files i /var/www/html/allsky.
- Renames NIGHTS_TO_KEEP to DAYS_TO_KEEP because it keeps/deletes a full day's worth of images, not just nighttime images.
Added new WEB_NIGHTS_TO_KEEP setting to config.sh that specifies how many days of images to keep in the website, if it's installed. This is similar to NIGHTS_TO_KEEP.
* Renamed "NIGHTS_TO_KEEP" to "DAYS_TO_KEEP" since it determines how many days' data is kept, not just nighttime data. * Removed the "AUTO_DELETE" variable; instead, if "DAYS_TO_KEEP" is "", then all days are kept, otherwise only the specified number of days are kept. This eliminates one variable while keeping the old functionality. * Added "WEB_DAYS_TO_KEEP" to clean out the Allsky website files.
* Renamed $NIGHTS_TO_KEEP to $DAYS_TO_KEEP since it applies to days, not just nights.
* Added ${ME} to log messages so it's clear where they are coming from.
* Removed $AUTO_DELETE since if $DAYS_TO_KEEP is "" ALL days are kept (same as old AUTO_DELETE=false).
thomasjacquin
approved these changes
Mar 20, 2022
config_repo/config.sh.repo
Outdated
Comment on lines
164
to
169
| # If you have additional data or buttons you want displayed on the WebUI's System page, | ||
| # specify one or more colon-separated files that contain that data. | ||
| # You must provide scripts to update those files. | ||
| # See the WEBUI_DATA_FILES configuration variable in the "Software settings" section of | ||
| # https://github.com/thomasjacquin/allsky/wiki/allsky-Settings for details. | ||
| WEBUI_DATA_FILES="" |
There was a problem hiding this comment.
Looks like you've based this branch out of another one instead of master but that shouldn't be a problem. You may end up with a merge conflict but it should resolve easily
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.