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(efb): fix checklists not turning green if completed but not in the relevant flight phase #8785

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Fabi-02
Copy link

@Fabi-02 Fabi-02 commented Jul 31, 2024

Fixes #8783

Summary of Changes

If checklists are completed, they are now turning green, even if the plane is not in the relevant flight phase.
Additionally, the checklists that have not been completed yet gets a bit faded if they belong to a non-relevant flight phase.

Screenshots

Screenshot 2024-07-31 104250
Screenshot 2024-07-31 104331

References

N/A

Additional context

Discord username (if different from GitHub): xaliburmc

Testing instructions

Check the correct behavior of the checklist tabs on the EFB.

How to download the PR for QA

Every new commit to this PR will cause new A32NX and A380X artifacts 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, find and click on the PR Build tab
  4. Click on either flybywire-aircraft-a320-neo or flybywire-aircraft-a380-842 download link at the bottom of the page

@alepouna alepouna added the EFB label Jul 31, 2024
@alepouna alepouna requested a review from Benjozork August 2, 2024 22:44
@2hwk
Copy link
Member

2hwk commented Aug 17, 2024

This feels like it could create more bugs than what it's attempting to fix, I'd caution that this would require a bit of testing.

@Fabi-02
Copy link
Author

Fabi-02 commented Aug 18, 2024

Could be. What was the initial reason for only highlighting the checklists of the relevant flight phase?

@2hwk
Copy link
Member

2hwk commented Sep 24, 2024

Approved for now code wise, but needs some scrutiny on testing.

EDIT: One modification could be to check if it is in a current or previous flight phase, but not a flight phase that has yet to be entered.

@2hwk 2hwk added A32NX Related to the A32NX aircraft A380X Related to the A380X aircraft QA Tier 1 QA Ready to Test labels Sep 24, 2024
@2hwk 2hwk added this to the v0.12.0 milestone Oct 19, 2024
@2hwk 2hwk added QA Tier 2 and removed QA Tier 1 labels Oct 19, 2024
@flogross89 flogross89 modified the milestones: v0.12.0, v0.13.0 Oct 26, 2024
.github/CHANGELOG.md Outdated Show resolved Hide resolved
@2hwk
Copy link
Member

2hwk commented Nov 16, 2024

Quality Assurance Tester Report

Discord Username: 2cas
Object of testing: #8785
Aircraft: A32NX
Tier of Testing: 1
Date: 16/11/2024

Testing Process:

  • Tested that checklists work in their intended flight phase
  • Tested that checklists work out of their intended flight phase

Taxi checklist unchecks itself on ground after turning off PRED WIND SHEAR.

image

Testing Results:
Not Passed

Negatives:

  • Doesn't appear to fix the bug listed and creates a new bug noted above.

Conclusions:

  • Back to the drawing board I think for this one.

@2hwk 2hwk marked this pull request as draft November 16, 2024 16:45
@2hwk 2hwk removed this from the v0.13.0 milestone Nov 16, 2024
@Benjozork
Copy link
Member

@Fabi-02, are you still planning to work on this?

@Benjozork Benjozork added the Waiting For Response This issue or PR needs a response from the author label Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A32NX Related to the A32NX aircraft A380X Related to the A380X aircraft EFB QA Failed QA Tier 1 Waiting For Response This issue or PR needs a response from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[EFB] Landing and After-Landing Checklist don't turn green until after landing
6 participants