Skip to content

fix: use home variable in script to reset uploads#99

Merged
gjreda merged 2 commits into
mainfrom
fix-reset-uploads-script
Jun 6, 2023
Merged

fix: use home variable in script to reset uploads#99
gjreda merged 2 commits into
mainfrom
fix-reset-uploads-script

Conversation

@sehyod

@sehyod sehyod commented Jun 6, 2023

Copy link
Copy Markdown
Collaborator

Fixes #98

  • use $HOME instead of ~ in the path to get the valid path
  • add -f to rm "$PROJECT_DIR/uploads/"*.pdf so that the script does not fail when the uploads directory is empty

@sehyod sehyod requested a review from cguedes June 6, 2023 09:32
cguedes
cguedes previously approved these changes Jun 6, 2023
@sehyod sehyod changed the title fix: path in script to reset uploads fix: use home variable in script to reset uploads Jun 6, 2023
@cguedes cguedes self-requested a review June 6, 2023 09:45
@gjreda gjreda merged commit 05fcde5 into main Jun 6, 2023
@gjreda gjreda deleted the fix-reset-uploads-script branch June 6, 2023 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

reset_uploads script does not work

3 participants