Compare commits

1 Commits
v1.0.0 ... main

Author SHA1 Message Date
Eric Wagoner
731e9ac58c Add ownership warning to deploy --reset-data
The script now prints the chown command needed after pushing data,
since uploaded files are owned by admin and the web server can't
write to them. Updated docs to match.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2025-12-24 14:40:41 -05:00
2 changed files with 7 additions and 3 deletions

View File

@@ -73,9 +73,9 @@ The `deploy` script uses rsync to push to YunoHost:
**Important deployment notes:**
- Files in `data/` must be owned by the web server user (`my_webapp__2`) for PHP to write to them
- If deploying with `--reset-data`, the uploaded files get admin ownership - the web server then can't edit them
- Fix with: `sudo chown my_webapp__2:my_webapp__2 /var/www/my_webapp__2/www/data/inventory.json`
- Adding admin to the my_webapp__2 group allows the deploy script to write while keeping web server access
- Using `--reset-data` uploads files as admin — you must fix ownership afterward (the script shows the command)
- If edits aren't saving, check ownership with `ls -la` and fix with:
`sudo chown -R my_webapp__2:my_webapp__2 /var/www/my_webapp__2/www/data/`
## Key Implementation Details

4
deploy
View File

@@ -16,6 +16,10 @@ if [ "$1" = "--reset-data" ]; then
echo "Pushing local data to server..."
rsync -avz --no-t --no-p \
data/ ${HOST}:${DIR}/data/
echo ""
echo "*** Data pushed. Fix ownership so the web server can write: ***"
echo " ssh ${HOST} \"sudo chown -R my_webapp__2:my_webapp__2 ${DIR}/data/\""
echo ""
else
echo "Pulling data from server..."
mkdir -p data