Skip to content

Script upgrade#10

Open
propersam wants to merge 2 commits intoabjennings:masterfrom
propersam:script_upgrade
Open

Script upgrade#10
propersam wants to merge 2 commits intoabjennings:masterfrom
propersam:script_upgrade

Conversation

@propersam
Copy link
Copy Markdown

Made the scripts compatible with both gmail and amazon work mail. I used it to download my email from Amazon Work mail. working fine and easy to use.

@abjennings
Copy link
Copy Markdown
Owner

Thanks for the contribution. I appreciate your attention to my project. Sorry for the slow reply. This spring has been crazy for me.

One of my goals for the project has been to keep it as small and simple as humanly possible. I want an average python programmer to be able to audit the code easily. So I'm thinking this merge commit that doubles the size of dobackup.py is not a good fit for the project.

Do you think we could keep dobackup.py as is, maybe rename it backup_gmail.py and make a copy called backup_amazonwork.py that is a simple amazon version?

If they have to specify things like an imap server and a folder, then I think I'd prefer to have them be command line arguments.

We may be able to work something out. Or you're welcome to fork it and go your own direction if you like.

@propersam
Copy link
Copy Markdown
Author

Hi there @abjennings Last year I was desperately in need of a free software i can use in backing up my aws mail so that i cn upload it gmail. My company was changing from aws to gsuite mail. I couldn't find any helpful software till i found yours but it was for gmail.
So i had to do necessary modifications nd make it work for awsmail. Which i then used to successfully back up all our mails and have them transferred to gsuite.
So i made a pull request just incase there's someone else like me later on.

The simplicity of ur code made it easier for me to understand and modify and i think it would be wise to keep it that way.
So yea, i agree with the idea of breaking the code down and making it as short as humanly possible.

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

Successfully merging this pull request may close these issues.

2 participants