flight-analysis icon indicating copy to clipboard operation
flight-analysis copied to clipboard

Check for colon for time when searching for time

Open djrcawley opened this issue 1 year ago • 1 comments

image

Check for colon when scraping and converting to date-time. Airline LATAM creates a Value Error when we check solely for 'AM' and 'PM'

djrcawley avatar Jun 09 '23 18:06 djrcawley

Hello,

I'm still pretty new to github and want to contribute where I can, so I apologize if my formatting is not the best. The issue I found, while very small, occurs anytime the airline LATAM is scraped. Since args checks for strings containing "AM" and formats them to a datetime. Instead, I added another check to verify if a colon was included in each string before it is formatted to a datetime.

Let me know if you have any other suggestions. I know this isn't a major issue, but hoped to still contribute anyways. @kcelebi

djrcawley avatar Jun 15 '23 23:06 djrcawley