-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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): flyPadOS v2 🚀 #4026
feat(efb): flyPadOS v2 🚀 #4026
Conversation
4a9d3df
to
13d2a47
Compare
f5ff1be
to
0431530
Compare
9e5d074
to
5b778ef
Compare
28a747d
to
560d314
Compare
Quality Assurance Tester Report Discord: Cdr_Maverick#6475 Testing Process:
Negatives:
Testing Results: Not Passed Conclusions: Media: |
This is expected and working as intended. |
The latest commit should fix the map panel and settings switches issue. Can someone test when it's done building ? |
Quality Assurance Tester Object of testing: #4026 Testing Process:
Negatives:
Testing Results: Not Passed due to the dispatch page problem, other than that i think its good to go Conclusions: (Notes made, the positives in the PR, and anything extra you would like to mention - Delete Line if Nothing Extra is to be mentioned |
Discord: easyjetsimpilot Testing Process: Full flight which tested all pages and functions of the EFB OS v.2 (with Navigraph). All functions worked as expected. Negatives:
Conclusions: Overall, everything works as it should - however, the 3 items above I feel need investigating - particularly the ability to maintain the pages of the VATSIM charts if navigating away from the page. Thanks for all the continued work. |
Hello. The ATC page is refreshing every two minutes. Wish is also the refresh rate of data coming from vatsim/Ivao. |
My page did not update at all when coming into my Destination airport. So that's like 20min without an update. Even changing pages did not do anything. |
Ok Thanks. I'll do some extra tests because I cannot reproduce it. I'll plan a new flight |
@St54Kevin . I spot the issue and fixed it. PR still need to be merge |
Quality Assurance Tester Report Testing Process:
Negatives:
Testing Results: (Not) Passed Hope this helps :-) |
* display page with the list of ATCs * code simplification * fix sorting issue * clean console.log * performance improvment * remove useless css file * add wrap for text atis * align with new api-client version * merge * merge * Revert "merge" This reverts commit db6ab62. * update api-client package * Update ATC.tsx * Revert "Update ATC.tsx" This reverts commit ef45e2b. * fix issue on atc list refresh * remove console.log * add atis controllers in ATC page * remove console.log * decrease refresh rate * fix refresh issue * remove double load * increase font-size * fix distance calculation * use useCallback and useInterval * add atisSource as dependency * try to repect rules of hook
Quality Assurance Tester Object of testing: #4026 Testing Process:
Negatives:
Testing Results: Passed. I couldnt reproduce the ground services bug on another airport |
Just tested with the same sim on a different airport (EDDW) and ground services just worked fine. Maybe that was due to the EGLL scenery. Changed qa to Passed |
Fixes: #4053
Fixes: #4075
Fixes: #4575
Important
Remove env file from PR workflow before merge.
Summary of Changes
flyPadOS v2. Full Layout & UI Rewrite. Special thanks to @Benjozork & @ZigTag!
Screenshots (if necessary)
Testing Instructions
Please thoroughly test every page (if you don't have a navigraph subscription don't worry about it).
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.