list-all-pages; mailgun lib replace

icebob / hasznaltauto-figyelo

Autó figyelő alkalmazás a hasznaltauto.hu oldalhoz

MIT License

30 stars 13 forks source link

list-all-pages; mailgun lib replace #7

Closed DBLaci closed 2 years ago

DBLaci commented 2 years ago

The changes I made:

mailgun lib is deprecated - I changed it to the official one - the configuration hasn't changed
we search all pages until empty found - this is the goal of the feature
minor changes in formatting and readme

DBLaci commented 2 years ago

FIrst: I updated the Dockerfile in this commit, because instead of clone I used add, but adding everything was not safe and sane, sorry.

Second: yes, I tested. Maybe the originally used lib wasn't bad, but I added wrong url to the config. BUT the old lib was wrong because it said the sending was fine - which was a lie... SO in the lib I added the errors are handled correctly... And yes it works for me!

icebob commented 2 years ago

Great, thanks!