search

tigattack / VeeamDiscordNotifications

Send Veeam Backup & Replication session summary notifications to Discord.
GNU General Public License v3.0
12 stars 3 forks source link

Add option to deploy post-script per job after install #20

Closed Philenst closed 3 years ago

Philenst commented 3 years ago

After the installation script, it'll grab all the jobs. If there isn't any. It'll print this and exit soon after. If there is, it'll iterate through all the jobs and ask the user if they want to overwrite the current script (If there's already one configured) or if they want to add a new one (If currently disabled).

I know the comments aren't the greatest therefore I'll gladly take an L to improve them if needed.

Philenst commented 3 years ago

Moved it around a bit. Added the Post Script "deployment" to a new file so if any new instances are ever added and/or file names are changed. It can be modified & ran again to update all jobs accordingly. AutoInstaller just calls the file.

Philenst commented 3 years ago

Current dev still uses DiscordNotificationBootstrap.ps1, I'm already moved to agent-support so I was coding it around Bootstrap.ps1, this is a fix for the right path for this branch.

tigattack commented 3 years ago

Phil, cheers for this. Before I review, could you please rebase your branch on dev?

I've improved the CI so this will help highlight anything which could be improved without the need for review.

EDIT: Closed upon request. Replaced by #22.