docker support with env variables support

[maxxxdj]

[maxxxdj]

Hey, I've ammended the readme change + the null check for the numeric variables. I also added an option if we pass only one user in the USERS_TO_FOLLOW section.

About the other changes -> i removed them, but i didn't succeed to squash the commits, hope you can do that. The other changes are important, related to puppeteer to be ran properly in a container.

Cheers!

[mifi]

thanks. I did some additional improvements. I still see that yarn.lock, package.json, index.js changes are part of this PR. I don't think they are needed? If not, could you revert those?

[maxxxdj]

thanks. I did some additional improvements. I still see that yarn.lock, package.json, index.js changes are part of this PR. I don't think they are needed? If not, could you revert those?

I've tried with the old ones, but Dockerfile was failing to build. I suggest to try to run it locally and if it's not a problem to leave them. Cheers!

[mifi]

That’s odd. Maybe try yarn add instuto instead of npm install instauto

[maxxxdj]

That’s odd. Maybe try yarn add instuto instead of npm install instauto

It worked! Great, thanks. The only changes now are some IDE or whitespaces. I've changed the docker container name to be the correct as your project and product - "instauto".

Cheers

[mifi]

thanks! i would appreciate if you also remove the whitespace changes. after that is solved i think this can be merged

[maxxxdj]

thanks! i would appreciate if you also remove the whitespace changes. after that is solved i think this can be merged

done!

[thegreenmilecornant]

Any news on this PR? Would love to use a simple docker compose to deploy instauto.

Tried building an image with the dockerfile (first time) - but failed.

[maxxxdj]

Any news on this PR? Would love to use a simple docker compose to deploy instauto.

Tried building an image with the dockerfile (first time) - but failed.

Hey! I'm not sure why the owner didn't merged this one. It was running fine on my side and even I had it with env variables so I can run different accounts. If you copied that file and trying it on the current master maybe won't work, due to changes in master branch.

Try to ping mifi

[mifi]

Hi. I left some code review questions that were not answered

[thegreenmilecornant]

Hey! I'm not sure why the owner didn't merged this one. It was running fine on my side and even I had it with env variables so I can run different accounts. If you copied that file and trying it on the current master maybe won't work, due to changes in master branch.

Try to ping mifi

OK. Would you be so kind and look at the requested changes by @mifi and update the PR 😃 ? Would love to deploy this in Docker!

[maxxxdj]

Hi. I left some code review questions that were not answered

So I hope you are asking about the CMD...

First of all I am not a node.js programmer, but from what I got from logs I saw that it needs an upgrade on the yarn dependency and pupeteer. This is why I added those arguments when executing the run command.

About COPY command - I am copying the root folder to docker image folder, from where I will execute the program.

[mifi]

if these changes look ok then we can merge

[mifi]

I haven't tested this but let me know if it doesnt work and we can do another PR to fix it