PR: Update documentation, make script file argument ordering more flexible

Author: jlchilders11

README.md

@@ -181,7 +181,7 @@ Large static images and other assets are stored in S3 buckets rather than in the
 ### Prerequisites
 
 - `awscli` must be installed.
-- Configure a set of credentials in your `~/.aws/credentials` file with the profile name `upload-images`:
+- Configure a set of credentials in your `~/.aws/credentials` file with the profile name `sync-boost-images`:
 
   ```ini
   [sync-boost-images]
@@ -207,8 +207,6 @@ To upload to all S3 buckets:
 $ just up_sync_images_all_buckets
 ```
 
-*Note: The script will prompt you for the S3 destination path, defaulting to `/static/img/v3`.*
-
 #### Downloading
 
 To download missing or outdated static items from the staging bucket to your local directory:
@@ -221,11 +219,11 @@ $ just down_sync_images
 
 ## Deploying
 
-TDB
+TBD
 
 ## Production Environment Considerations
 
-TDB
+TBD
 
 ---
 

core/templatetags/custom_static.py

@@ -9,7 +9,7 @@ def large_static(file_path: str):
     """
     Automatically handles serving large static content based on development vs. production.
 
-    Serves specified content from the local large static file is LOCAL_DEVELOPMENT is True, else serves
+    Serves specified content from the local large static file if LOCAL_DEVELOPMENT is True, else serves
     from the static content in S3.
     """
     # Strip any leading slashes to avoid footguns

justfile

@@ -2,7 +2,6 @@ set dotenv-load := false
 COMPOSE_FILE := "docker-compose.yml"
 ENV_FILE := ".env"
 DJANGO_VERSION := "5.2"
-VALID_BUCKETS := "boost.org.v2 stage.boost.org.v2 boost.org-cppal-dev-v2"
 
 
 @_default:

scripts/sync-large-static-images.sh

@@ -6,8 +6,7 @@ set -euo pipefail
 DEFAULT_BUCKET="stage.boost.org.v2"
 S3_BUCKETS="boost.org.v2 boost.org-cppal-dev-v2 ${DEFAULT_BUCKET}"
 AWS_PROFILE='sync-boost-images'
-DEFAULT_DEST="/static/"
-DEST_PATH=${DEFAULT_DEST}
+DEST_PATH="/static/"
 
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 SOURCE_DIR="$(dirname "$SCRIPT_DIR")/static/static-large/"
@@ -28,7 +27,7 @@ Options:
   --help           Display this help and exit.
 
 Configuration:
-  The default destination for upload is ${DEFAULT_DEST}.
+  The default destination for upload is ${DEST_PATH}.
   In your .aws/credentials file, add a set of credentials:
 
   [${AWS_PROFILE}]
@@ -123,6 +122,8 @@ download_images() {
 
 
 ALL_BUCKETS=false
+UPLOAD_COMMAND=false
+DOWNLOAD_COMMAND=false
 while [[ $# -gt 0 ]]; do
   case "$1" in
     --help)
@@ -133,12 +134,10 @@ while [[ $# -gt 0 ]]; do
       ALL_BUCKETS=true
       ;;
     --up-sync)
-      upload_images "$ALL_BUCKETS"
-      exit $?
+      UPLOAD_COMMAND=true
       ;;
     --down-sync)
-      download_images
-      exit $?
+      DOWNLOAD_COMMAND=true
       ;;
     *)
       echo "Unknown option: $1"
@@ -148,3 +147,18 @@ while [[ $# -gt 0 ]]; do
   esac
   shift
 done
+
+if [ "${UPLOAD_COMMAND}" == true ] && [ "${DOWNLOAD_COMMAND}" == true ]; then
+  echo "Uploading and downloading must be done seperately. Exiting.";
+  exit 0;
+elif [ "${UPLOAD_COMMAND}" == true ]; then
+  upload_images "$ALL_BUCKETS";
+  exit $?;
+elif [ "${DOWNLOAD_COMMAND}" == true ]; then
+  download_images;
+  exit $?;
+else
+  echo "Unknown command";
+  usage;
+  exit 1;
+fi