[548] workflow_dispatch boolean inputs are not actually booleans (#550)

* Workflow_dispatch inputs bools are strings and not booleans * Add a comment as to why || was added * revert the || change as that's not useful --------- Co-authored-by: Phil Nachreiner <[email protected]>
philosowaffle · Oct 9, 2023 · 2a60f6d · 2a60f6d
1 parent 1808577
commit 2a60f6d
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/.github/workflows/sync_peloton_to_garmin.yml b/.github/workflows/sync_peloton_to_garmin.yml
@@ -90,14 +90,15 @@ jobs:
         TZ: America/Chicago
 
     - name: archive files
-      if: ${{ github.event.inputs.saveLocalCopy }}
+      # saveLocalCopy can be set when workflow action is manually run otherwise it's always skipped 
+      if: ${{ github.event.inputs.saveLocalCopy == 'true' }}
       run: |
         apt-get update
         apt-get install -y zip
         zip -r -j output.zip ${{ env.OUTPUT_DIR }}
         
     - uses: actions/upload-artifact@v3
-      if: ${{ github.event.inputs.saveLocalCopy }}
+      if: ${{ github.event.inputs.saveLocalCopy == 'true' }}
       with:
         name: output
         path: output.zip