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(ISIS): removes ability to change a bug if selected item is Off #7280

Merged
merged 3 commits into from
Jun 10, 2022

Conversation

patmack14
Copy link
Contributor

@patmack14 patmack14 commented Jun 6, 2022

Fixes #[issue_no]

Summary of Changes

This issue was brought to my attention by @tracernz. When the ISIS bug selectors, it previously allowed you to set a bug that was "off" or not activated. This fix removes the ability to set a bug when said item is in the "off" state.

Screenshots (if necessary)

References

https://youtu.be/U1bCkxL1QmY?t=69
https://youtu.be/NvKIk8dQnvs?t=325

Additional context

Discord username (if different from GitHub):
@pat M

Testing instructions

Open the ISIS Bugs page and attempt to set a bug that is in the "OFF" state.

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

@2hwk 2hwk added the QA Tier 1 label Jun 6, 2022
@@ -31,6 +31,9 @@ export const ISISDisplay: React.FC = () => {

useInteractionEvent('A32NX_ISIS_KNOB_CLOCKWISE', () => {
lastPilotInput.current = Date.now();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For anyone else reviewing, the XML only sends the event if bugs mode is active... 😅

@marcman86
Copy link
Contributor

Quality Assurance Report

Discord: marcman86#4907
Object of testing: #7280
Tier of Testing: 1
Date: 6/06/2022

Testing Process:

  • Spawned C&D at EDDP
  • Turned on bat. and ext. power
  • Test ISIS bugs

Negatives: Bugs values flicker when changed (click and scroll). Not of this PR, same in experimental version.

Testing Results: Passed
Conclusions: Works for me.

Hope this helps :-)

2hwk added a commit that referenced this pull request Jun 6, 2022
@2hwk 2hwk added the Exp Available on experimental branch (for testing) label Jun 7, 2022
2hwk added a commit that referenced this pull request Jun 7, 2022
@mico975
Copy link
Contributor

mico975 commented Jun 8, 2022

Quality Assurance Tester Report

Discord : mico#3145
Object of testing: #7280
Tier of Testing : 1
Date : 08/06/2022

Testing Process:
Open the ISIS Bugs page and attempt to set a bug that is in the "OFF" state. - OK, doesn't work when OFF

Negatives:
N/A

Testing Results:
Passed

Conclusions:
LGTM

@beheh
Copy link
Member

beheh commented Jun 8, 2022

@patmack14 Could you please still add an entry to CHANGELOG.md?

@patmack14
Copy link
Contributor Author

@patmack14 Could you please still add an entry to CHANGELOG.md?

Doh! (I knew I forgot somethin!) Will update it sometime tonight/early tomorrow.

@2hwk 2hwk added this to the v0.9.0 milestone Jun 9, 2022
@patmack14 patmack14 closed this Jun 9, 2022
@patmack14
Copy link
Contributor Author

@patmack14 Could you please still add an entry to CHANGELOG.md?

Done

@BlueberryKing BlueberryKing reopened this Jun 9, 2022
@2hwk 2hwk merged commit 0b69594 into flybywiresim:master Jun 10, 2022
@2hwk 2hwk removed the Exp Available on experimental branch (for testing) label Jun 10, 2022
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.

7 participants