Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Test email configuration in the pre-check #72

Open
VARGA-Peter opened this issue Apr 21, 2018 · 4 comments
Open

Feature request: Test email configuration in the pre-check #72

VARGA-Peter opened this issue Apr 21, 2018 · 4 comments

Comments

@VARGA-Peter
Copy link

Guys, great you have done here! This is finally the backup solution I was looking for. Thx to Tobias for the link in the Citrix forum.

May be you can re-consider the email treatment. It is very important for scheduled tasks to be informed.
Could you extract the email-settings from the python source into the .cfg file and add a command line option which tests the email settings with a test email?

Kind regards and thank you so much!

@VARGA-Peter VARGA-Peter changed the title Email support Feature: Test email configuration in the pre-check Apr 22, 2018
@VARGA-Peter VARGA-Peter changed the title Feature: Test email configuration in the pre-check Feature request: Test email configuration in the pre-check Apr 22, 2018
@NAUbackup
Copy link
Owner

Peter,
Any code changes you wish to submit for incorporation will certainly be taken into consideration and though these days I am limited in the time I can invest as I lost a co-worker who was helping with some development a few months ago, I will strive to do what I can. Thanks!

--Tobias

@VARGA-Peter
Copy link
Author

VARGA-Peter commented Apr 22, 2018

If you don't mind I can try to program this pre-check and submit it to you via email. You can then decide if it meets your expectations - I never programmed in Python but I am a programmer for 30 years so I should be capable to do it.

But please understand should it be programmed too complicated due to missing Python knowledge...

OR

When you decide to add #74 to the package then it would be already done because my example is totally stand alone. The user has only to copy the changes from the test file into VmBackup.py

Just let me know...

@NAUbackup
Copy link
Owner

NAUbackup commented Apr 23, 2018

Hi, Peter:
I should be able to incorporate that code as a module pretty easily. I try to incorporate multiple features before releasing an official new release, but would be more than happy to send you a preview version to test if you'd be willing to give it a go. I'm not sure what my load will be this week overall, but let me see what I may be able to accomplish. Thanks for your feedback -- it is truly appreciated. I just got back into town from a meeting, and will see if I can at least give things a quick review tomorrow to get an idea of the scope. Injecting code isn't always so easy if it has to integrate within the current logic flow. We'll see how it looks like in terms of time resources.
--Tobias

@NAUbackup
Copy link
Owner

Will try to tackle this next. Putting email configurations optionally into the configuration file makes good sense.

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

No branches or pull requests

2 participants