search

dizda / CloudBackupBundle

Be able to backup your database(s) and upload it to the cloud (Dropbox, Amazon S3, GoogleDrive, etc.)
MIT License
196 stars 57 forks source link

Suggestions to improvements #53

Closed Nyholm closed 9 years ago

Nyholm commented 9 years ago

Reply to #51

I think we should not bother with mails. We write logs instead. If you want to have a mail, config your monolog.

I also simplified stuff a bit. I consider the ClientChain as a normal Client. Same with the DatabaseChain.

Did also fix the DatabaseInterface. No need to have getCommand public.

dizda commented 9 years ago

Alright, that's good suggestions!

Thanks @Nyholm ! :+1:

Nyholm commented 9 years ago

I will also rewrite some stuff in the processors.. Give me a sec.

dizda commented 9 years ago

Ok