samyun / southwest-price-drop-bot

Bot that watches Southwest flights for price drops.
Other
74 stars 41 forks source link

Added support for number of passengers #63

Closed brussrus closed 2 years ago

brussrus commented 4 years ago

I did a recent fork and included some basic support for including the number of passengers to each of your flight alerts. I find this helpful when I'm looking at different flights for other people or for different trips where the number of people flying is different.

I did not account for existing flights you may already be tracking. That is, this could break those flights as I didn't include any logic for a previous entry in the database that doesn't include a passenger field. I just deleted the few flights I had and recreated them.

A few added notes: I find running with CHROME_DEBUG env variable set to TRUE gives me the fewest issues. I'm running this on my own machine, therefore not Heroku, etc. as a scheduled task and I've had no issues with running it this way. When I don't use the debug it often times doesn't want to load the page and pull prices appropriately. Not certain why running in debug would be different.

Also, I updated puppeteer-extra-plugin-stealth module and maybe a couple of others (acorn, react) to the latest versions. I found there were fewer errors displayed when I ran this from a console. after doing this.

Hopefully, you can include some of these changes in your version.