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

feat(fm): ils freq/ident on arrival page #8254

Merged
merged 3 commits into from
Oct 20, 2023

Conversation

tracernz
Copy link
Member

@tracernz tracernz commented Oct 16, 2023

Fixes #8253

Summary of Changes

Show the ILS ident and frequency for ILS approaches on the arrivals pages.

Note: Some 3rd party airports break the link between the ILS in the navdata and the runway.. in that case the ILS details will not be shown on the departure or arrivals pages. (fix for the scenery devs https://docs.flightsimulator.com/html/Content_Configuration/Environment/Airports_And_Facilities/Runway_Definition_Properties.htm#h8)

Screenshots (if necessary)

image

References

Some private photographs from a Honeywell H3 box.

Additional context

Discord username (if different from GitHub):

Testing instructions

Check that the ILS frequency and ident show on ILS approaches and not on any other types.

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, click on the bottom PR tab
  4. Click on the A32NX download link at the bottom of the page

@tracernz tracernz added this to the v0.11.0 milestone Oct 16, 2023
@Saschl
Copy link
Contributor

Saschl commented Oct 19, 2023

QA Report
Tester: saschl

Test report:

  1. Loaded several airports with and without ILS approaches
  2. Checked if ILS ident is present -> passed (EDDS, EDSB, LEPA, EGCC, KFJK)..

image

Result:

Passed

@tracernz tracernz enabled auto-merge (squash) October 20, 2023 03:28
@tracernz tracernz merged commit 112c19e into flybywiresim:master Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✔️ Done
Development

Successfully merging this pull request may close these issues.

MCDU: F-PLN arrival page does not display ILS information if available
2 participants