Remove explanatory comment

Back up snappdf's Chromium download to avoid re-fetching on every update
vendor/bin/snappdf download is idempotent: it exits immediately without downloading if versions/revision.txt already exists, and PDF generation doesn't check that version against the installed snappdf package - it just uses whatever's in versions/. So the cached Chromium build survives just fine across an update via the existing backup/restore mechanism. Add vendor/beganovich/snappdf/versions to create_backup/restore_backup; keep the snappdf download call only as a safety net for when the backup doesn't have it yet (e.g. the first update after this fix ships).
2026-07-22 05:42:53 +02:00 · 2026-07-21 23:08:21 +02:00 · 2026-07-21 23:03:53 +02:00 · 2026-07-21 22:58:31 +02:00
3 changed files with 9 additions and 3 deletions
@@ -35,12 +35,18 @@ function update_script() {
    systemctl stop supervisor nginx php8.4-fpm
    msg_ok "Stopped Services"

-    create_backup /opt/invoiceninja/.env /opt/invoiceninja/storage /opt/invoiceninja/public/storage
+    create_backup /opt/invoiceninja/.env /opt/invoiceninja/storage /opt/invoiceninja/public/storage /opt/invoiceninja/vendor/beganovich/snappdf/versions

    CLEAN_INSTALL=1 fetch_and_deploy_gh_release "invoiceninja" "invoiceninja/invoiceninja" "prebuild" "latest" "/opt/invoiceninja" "invoiceninja.tar.gz"

    restore_backup

+    msg_info "Verifying Chromium for PDF Generation"
+    cd /opt/invoiceninja
+    $STD ./vendor/bin/snappdf download
+    chown -R www-data:www-data /opt/invoiceninja/vendor/beganovich/snappdf/versions
+    msg_ok "Verified Chromium for PDF Generation"
+
    msg_info "Running Migrations"
    cd /opt/invoiceninja 
    $STD php artisan migrate --force
@@ -43,7 +43,7 @@ function update_script() {

    msg_info "Rebuilding Application"
    cd /opt/nametag
-    $STD npm ci --include=dev
+    $STD npm ci
    set -a
    source /opt/nametag/.env
    set +a
@@ -20,7 +20,7 @@ fetch_and_deploy_gh_release "nametag" "mattogodoy/nametag" "tarball" "latest" "/

 msg_info "Setting up Application"
 cd /opt/nametag
-$STD npm ci --include=dev
+$STD npm ci
 DATABASE_URL="postgresql://${PG_DB_USER}:${PG_DB_PASS}@127.0.0.1:5432/${PG_DB_NAME}" $STD npx prisma generate
 DATABASE_URL="postgresql://${PG_DB_USER}:${PG_DB_PASS}@127.0.0.1:5432/${PG_DB_NAME}" $STD npx prisma migrate deploy
 msg_ok "Set up Application"