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

fix(pilots-corner): replace SET MANAGED SPEED with CHECK SPEED MODE #894

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

BlueberryKing
Copy link
Member

Summary

Newer EIS standards change the wording from "SET MANAGED SPEED" to "CHECK SPEED MODE". Ideally, there would also be an annotation that this message is not implemented yet. However, flybywiresim/aircraft#8205 will implement it, so I think we can just leave it as is.

Location

https://docs.flybywiresim.com/pilots-corner/a32nx-briefing/pfd/special-message-annunciators/#set-managed-spd

Discord username (if different from GitHub): BlueberryKing

@vercel
Copy link

vercel bot commented Sep 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 1:45am

@Valastiri Valastiri added the waiting on a32nx PR PRs waiting for the corresponding a32nx repo PR to be merged label Sep 20, 2023
@Valastiri
Copy link
Member

@BlueberryKing just let us know when ur PR is merged -- alternatively you can set the Approved label here in this PR and it will run checks + merge for ya :)

@BlueberryKing BlueberryKing added Approved and removed waiting on a32nx PR PRs waiting for the corresponding a32nx repo PR to be merged labels Oct 25, 2023
@github-actions github-actions bot removed the Review Required PR Check Label label Oct 25, 2023
@github-actions
Copy link

Strict Build Failed! ❌ - Please checkout this PR locally to fix the issue before merging or see checks.

1 similar comment
@github-actions
Copy link

Strict Build Failed! ❌ - Please checkout this PR locally to fix the issue before merging or see checks.

@github-actions github-actions bot added Review Required PR Check Label and removed Approved labels Oct 25, 2023
@github-actions
Copy link

Strict Build Failed! ❌ - Please checkout this PR locally to fix the issue before merging or see checks.

@github-actions github-actions bot removed the Review Required PR Check Label label Oct 26, 2023
@github-actions github-actions bot merged commit 1cf8fe0 into primary Oct 26, 2023
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants