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(efb): Set default efb auto brightness on #9632

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrJigs7
Copy link

@MrJigs7 MrJigs7 commented Dec 6, 2024

Fixes #9630

Summary of Changes

Updated EFB so auto brightness defaults to on

Screenshots (if necessary)

References

Additional context

Discord username (if different from GitHub):
MrJigs

Testing instructions

  1. Load aircraft
  2. Power on EFB if necessary, check auto brightness is set to on in Flypad settings and quick settings

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, flybywire-aircraft-a380-842 (4K) or flybywire-aircraft-a380-842 (8K) download link at the bottom of the page

@heclak
Copy link
Contributor

heclak commented Dec 6, 2024

as part of the testing process, is there a need/steps to reset a user's settings? Or does a new build not read the existing settings? Curious if the testing is still valid if someone had already used a FBW aircraft and the old settings are already there.

@MrJigs7
Copy link
Author

MrJigs7 commented Dec 6, 2024

A new build does read existing settings as far as I’m aware. I tested this by loading the build and confirming the settings related to auto EFB brightness were applied on loading. I then set them to off, quit the flight, loaded back in and confirmed the settings were still set to off and user preference was therefore saved.

@heclak
Copy link
Contributor

heclak commented Dec 6, 2024

I meant if I already used another build of the A380 and had the auto-brightness set to on. How would i know if the auto-brightness on is because of your code change, or because of my previous setup. Just bringing this up to consider adding more details/steps in the testing steps.

If the EFB does read a users settings from another build, you might need to include testing steps to delete the previous saved config on their system before this test.

@tracernz
Copy link
Member

tracernz commented Dec 7, 2024

Why does this touch the large files repo?

For the setting, you can delete that specific key from the data store in the Coherent debugger DeleteStoredData(key: string), so DeleteStoredData('EFB_USING_AUTOBRIGHTNESS').

@MrJigs7
Copy link
Author

MrJigs7 commented Dec 7, 2024

Why does this touch the large files repo?

For the setting, you can delete that specific key from the data store in the Coherent debugger DeleteStoredData(key: string), so DeleteStoredData('EFB_USING_AUTOBRIGHTNESS').

Apologies I think I accidentally committed all from both my aircraft source repository and my large files repository. Should I make a new commit without the large files changes?

@MrJigs7 MrJigs7 force-pushed the Set_Default_EFB_Auto_Brightness_On branch from a546da2 to 9799bd6 Compare December 8, 2024 20:49
@MrJigs7 MrJigs7 force-pushed the Set_Default_EFB_Auto_Brightness_On branch from 9799bd6 to c33c46a Compare December 8, 2024 20:55
@MrJigs7
Copy link
Author

MrJigs7 commented Dec 8, 2024

@tracernz I've tidied this PR up by removing the unnecessary commits including the accidental commit relating to the large files repo

@Benjozork Benjozork added the EFB label Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 🟡 Code Review: Ready for Review
Development

Successfully merging this pull request may close these issues.

[EFB] Auto-Brightness Should be Default On
4 participants