-
Notifications
You must be signed in to change notification settings - Fork 18
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
Move to VATSIM API v2 #808
Comments
@blt950 Still waiting for VATSIM API key for v2. When obtained, I will jump onto it. |
Could be we need to support both v1 and v2 in a transition period to not block others of upgrading. |
Partially solves #808 by integrating VATSIM API Core v2 support. Co-authored-by: Daniel <[email protected]>
Roster is in place and rating times. What's left is the activity counter. In v1 we could restrict it to only return sessions from a given date to today, but now that's not possible - we can only limit amount in the reply. Waiting reply from VATSIM Tech if this will be implemented, otherwise we need to just ask for whole log and go through a long history and filter the dates as well ... |
@blt950 Thanks! I just updated necessary parts to v2 (which require auth) and left the rest as I've found out same problem with the filters as you. Tech team seems busy, so it was on purpose. But thank you anyway. If you want me to finish the rest when API Core is ready, just let me know. (First PR was a little wild from my side as I am not active PHP developer anymore and I am used to be controller by PHP PSRs, linters and static analyzers which have totally different setup than used in CC. I will do my best in future PRs :) ) |
Thanks anyway. It's not that urgent, so I guess we'll just await this last change until they implement it. Until then the v1 API just for this call specifically works fine, and is better than doing a temporary workaround due to missing functionality. |
VATSIM is no longer issuing v2 keys. Update our API calls to thew new API.
fetchApiUsers()
inUserController
UpdateAtcHours.php
UserController
andTrainingController
(awaiting confirmed implementation of needed timespan argument from VATSIM Tech)TrainingController:https://api.vatsim.net/api/ratings/1751077/rating_times/
->https://api.vatsim.net/v2/members/1751077/stats
The text was updated successfully, but these errors were encountered: