Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds the timestamp of the first fix. #2071

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vicb
Copy link
Contributor

@vicb vicb commented Jul 29, 2020

No description provided.

@vicb vicb marked this pull request as draft July 29, 2020 16:55
@hammed
Copy link

hammed commented Sep 26, 2023

Hi, please add this simple change to the API. It would make things a LOT easier for us and we won't have to create hacky workarounds to determine the flight date and full timestamps 🙏
The addition of the timestamp to the JSON response should have no impact on existing users of the API.

@hammed
Copy link

hammed commented Sep 27, 2023

🙏 Thank you @vicb for this change.

@vicb
Copy link
Contributor Author

vicb commented Sep 27, 2023

Oh 3 years ago, I forgot about that!

It's a shame this project only has minimal maintenance because it's quite nice otherwise.

Anyway if you need help with the hacky workarounds, I can share my code.

@hammed
Copy link

hammed commented Oct 3, 2023

Thanks Vicb I found your FlyXC project. Nice!

@vicb
Copy link
Contributor Author

vicb commented May 17, 2024

Do you think we can move this forward?

@hammed
Copy link

hammed commented May 17, 2024

Yes please! This simple change will make things a lot simpler for API users esp those in other time zones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants