Can someone see any obvious reason my plist isn't trigger scripts correctly?

I have this .plist called update.plist and I wan't it to trigger a shell script called weekly_update_trigger.sh every minute (changing to weekly after testing) and this is what the plist looks like:

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
        <key>Label</key>
        <string>update</string>
        <key>ProgramArguments</key>
        <array>
                <string>sudo</string>
                <string>sh</string>
                <string>/Library/JamfScripts/weekly_update_trigger.sh</string>
        </array>
        <key>StartInterval</key>
        <integer>60</integer>
</dict>
</plist>

Is there something obvious that I'm missing or that I should be looking for? The script isn't triggering, however when i run the script locally, it works.

The script is as such

#!/bin/bash
sudo /usr/local/bin/jamf policy -trigger weeklyupdate
exit 0