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.

Sign up for GitHub

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

Jump to bottom

Add AIRAC airport file key and system command #793

Closed
erikquinn opened this issue Oct 18, 2017 · 3 comments
Closed

Add AIRAC airport file key and system command #793

erikquinn opened this issue Oct 18, 2017 · 3 comments
Assignees
@eliuuk
Labels
feature Addition of a COMPLETELY NEW feature good first issue

Comments

@erikquinn
Copy link
Member

We need a way to share with the users the AIRAC version of the airport file they're using, or at least to record it for our own purposes of keeping them fairly up to date.

We can add a airac key to all airport files (and airport format documentation!) that we store in the AirportModel. A simple system command could then output the AIRAC number in use for the given airport. That way, if users are confused about why they can't use the procedures from current charts, we can all see the chart sets with which the airport is currently equipped. 😃

More than anything, I think this will do a lot for our sanity in maintaining our library of airports. When we see an airport still rocking AIRAC 1704 and the current is 1711, we will 😬 and 🔧 it!
@erikquinn erikquinn added the feature Addition of a COMPLETELY NEW feature label Oct 18, 2017
@eliuuk
Copy link
Member

eliuuk commented Oct 18, 2017

Excellent! 😁

@Fechulo
Copy link
Member

Fechulo commented Oct 18, 2017

Sounds good! I like it 👍

@filipjonckers
Copy link

good point - a simple "airac": "1711" will do

@eliuuk eliuuk self-assigned this Jan 21, 2018
@erikquinn erikquinn added the has pull request Add this label when an issue has a PR which will resolve it label Oct 20, 2018
@clscott42 clscott42 mentioned this issue Oct 20, 2018
Merged
@erikquinn erikquinn mentioned this issue Oct 30, 2018
Merged
@n8rzz n8rzz removed the has pull request Add this label when an issue has a PR which will resolve it label Nov 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eliuuk eliuuk

Labels
feature Addition of a COMPLETELY NEW feature good first issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@felixscheffer @n8rzz @filipjonckers @erikquinn @Fechulo @eliuuk