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

Define U.A.E FIR Approach Areas #393

Merged
merged 8 commits into from
Nov 3, 2024
Merged

Conversation

luke11brown
Copy link
Contributor

Description

Adds definitions for the following approach/departure positions:

  • Abu Dhabi (OMAA)
  • Dubai (OMDB)
  • Fujairah (OMFJ)
  • Ras Al Khaimah (OMRK)
  • Al Ain (OMAL)

Motivation and Context

No previous definitions

How to prove the effect of this PR?

OMAA
image

OMDB
image

OMFJ
image

OMRK
image

OMAL
image

All areas compound image
image

Additional info

Review from @DedpewlControl or @Ranbi2Delta

Is this a breaking change?

No

Checklist

  • My change or addition follow the formatting standard of the project.
  • I am on the list of approved contributors.

@dirtyformal
Copy link
Collaborator

Hi there @luke11brown, can you please follow the approval guidelines for contributors as stated in the repository readme? After that, I will review your PR.

@luke11brown
Copy link
Contributor Author

Hi, I'm not vACC staff, hence why I noted review will be from vACC staff @DedpewlControl or @Ranbi2Delta

@dirtyformal
Copy link
Collaborator

Hi, I'm not vACC staff, hence why I noted review will be from vACC staff @DedpewlControl or @Ranbi2Delta

Cool - get them to email me please.

@DedpewlControl
Copy link

Hello! Email sent from my side!

@Ranbi2Delta
Copy link

Ranbi2Delta commented Oct 28, 2024 via email

@luke11brown
Copy link
Contributor Author

Noted around Argoob that the centre of the arc was used instead of the circumference, now fixed.

image

@dirtyformal dirtyformal merged commit 7382f20 into vatsimnetwork:main Nov 3, 2024
2 checks passed
@dirtyformal
Copy link
Collaborator

Merged!

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

Successfully merging this pull request may close these issues.

4 participants