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): flyPadOS v2 🚀 #4026

Merged
merged 68 commits into from
Jul 12, 2021
Merged

feat(efb): flyPadOS v2 🚀 #4026

merged 68 commits into from
Jul 12, 2021

Conversation

hiaaryan
Copy link
Contributor

@hiaaryan hiaaryan commented Mar 19, 2021

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!

  • Add New Layout.
  • Add Navigraph Support.
  • Enable Remaining Settings Options.
  • Finishing Touches.
  • Remove INOP Pages.
  • Adjust All Pages.

Screenshots (if necessary)

image

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.

  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

@hiaaryan hiaaryan closed this Mar 19, 2021
@ZigTag ZigTag reopened this Mar 19, 2021
@ZigTag ZigTag marked this pull request as ready for review March 19, 2021 06:20
@ZigTag ZigTag marked this pull request as draft March 19, 2021 06:39
@ZigTag ZigTag marked this pull request as ready for review March 19, 2021 06:44
@hiaaryan hiaaryan marked this pull request as draft March 19, 2021 07:18
@hiaaryan hiaaryan added the EFB label Mar 19, 2021
@hiaaryan hiaaryan force-pushed the flypados-v2 branch 2 times, most recently from 4a9d3df to 13d2a47 Compare March 22, 2021 08:43
@Benjozork Benjozork added this to the v0.6.0 milestone Mar 22, 2021
@derl30n derl30n linked an issue Mar 22, 2021 that may be closed by this pull request
@ZigTag ZigTag force-pushed the flypados-v2 branch 2 times, most recently from f5ff1be to 0431530 Compare March 26, 2021 21:59
@Benjozork Benjozork changed the title feat(efb): FlyPadOS v2 🚀 feat(efb): flyPadOS v2 🚀 Apr 3, 2021
@hiaaryan hiaaryan marked this pull request as ready for review April 3, 2021 11:10
@hiaaryan hiaaryan marked this pull request as draft April 3, 2021 16:40
@Benjozork Benjozork modified the milestones: v0.6.0, v0.7.0 Apr 4, 2021
@hiaaryan hiaaryan force-pushed the flypados-v2 branch 3 times, most recently from 9e5d074 to 5b778ef Compare April 10, 2021 19:09
@hiaaryan hiaaryan force-pushed the flypados-v2 branch 6 times, most recently from 28a747d to 560d314 Compare April 19, 2021 16:05
@frankkopp
Copy link
Member

Quality Assurance Tester Report

Discord: Cdr_Maverick#6475
Object of testing: #4026
Tier of Testing: 1
Date: 21.6.2021

Testing Process:
C&D at EDDF
Simbrief flightplan EDDF-EDDM
Checking each page and subpage
Dashboard

  • Simbrief import ==> OK [1]
    Dispatch
  • OFP ==> OK (EZY format)
  • Fuel ==> OK (Defueling, Fueling)
    Ground
  • Check all button functions
    Performance
  • TOD: Ground Speed - see negatives ==> otherwise OK
  • Landing: see negatives ==> otherwise OK
  • Navigraph; ==> see negatives
  • ATC ==> OK
    Settings ==> OK

Negatives:

  • General:
    • Last subpage is not remembered - when changing to another page and back the first subpage is visible and not the last used [3] (this is annoying when for example on Landing performance jumping to Dashboard or OFP to lookup values)
  • Direct after starting tablet: Map icons transparent [0]
  • Link MCDU - not working / unclear what it should do
  • Ground Speed - Min Alt looks funny - can't edit (probably not meant to be editable) [2]
  • Landing: Metar does not fetch Wind Magnitude [4]
  • Landing: Clear does not work [4]
  • Navigraph:
    • Insufficient .env file - unclear how to access Navigraph subscription (self compiled) - Works when run via PR artefact [5]
    • looses selection when changing to other page and back
    • List of charts - can't scroll with mouse wheel
  • ATC:
    • ATIS font too small [6]
    • no ATIS when using PR artefact / worked with self-compiled build [6][7]

Testing Results: Not Passed

Conclusions:
Great feature but still too many small issues imho.
Also there seem to be differences when compiled locally vs. PR build

Media:
[0] Map icons transparent
image
[1] Simbrief OFP
image
image
[2]
image
[3]
https://user-images.githubusercontent.com/16833201/122691149-1c4bf280-d22e-11eb-9b5e-6e186755424b.mp4
[4]
https://user-images.githubusercontent.com/16833201/122691409-c8daa400-d22f-11eb-9188-9e3b100f1eb4.mp4
[5] Navigraph
image
[6] ATC
image
[7]
image

@Benjozork
Copy link
Member

Benjozork commented Jun 21, 2021

Insufficient .env file - unclear how to access Navigraph subscription (self compiled) - Works when run via PR artefact [5]

This is expected and working as intended.

@frankkopp

@Benjozork
Copy link
Member

Benjozork commented Jun 21, 2021

The latest commit should fix the map panel and settings switches issue.

Can someone test when it's done building ?

@St54Kevin
Copy link
Contributor

St54Kevin commented Jun 21, 2021

Quality Assurance Tester

Object of testing: #4026
Tier of Testing: 2
Date: 21/06/2021

Testing Process:

  1. Loaded C&D an setup Settings page like I want it to, calibrated throttles -> Check issues
  2. Checked dispatch page and refueled -> Check issues
  3. Tested all ground services and pushback
  4. Checked ATC page on this VATSIM flight
  5. I do not have a navigraph charts sub
  6. Played with ToD and LDG Calculators inflight

Negatives:

  • PTU toggle in settings page does not move, it just changes color
  • Something was wrong with the Dispatch page. I had no option to load my simbrief FPL. Username was set in the Settings menu. Consequently i also couldnt test OFP page
  • I also undestand the last subpage not being remembered may be a little annoying
  • ATC page does not update, when staying on it. But changing to another page and back to ATC, frequencies will update
  • ATC page didnt give me the TWR, GND and ATIS of LOWW although being within 50nm of the airport on approach

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
Media:
image
image
image
image

@easyjetsimpilot
Copy link

easyjetsimpilot commented Jun 21, 2021

QA1
Quality Assurance Tester

Discord: easyjetsimpilot
Object of testing: #4026
Tier of Testing: 2
Date: 21/06/21

Testing Process: Full flight which tested all pages and functions of the EFB OS v.2 (with Navigraph). All functions worked as expected.

Negatives:

  1. OFP zoom/page not maintained when navigating away from the OFP page
  2. Navigraph charts & selected airport not maintained when navigating away from the OFP page. Personally, I see this a a major issue. As you have to pop-out the EFB to type in the ICAO code (due to keybindings), this is time consuming if has to be done every time.
  3. VATSIM page - not all available controllers were shown as available on the page when they were online (according to VPilot) - image above
    Testing Results: Not Passed

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.

@ZeroKaa
Copy link
Contributor

ZeroKaa commented Jun 21, 2021

Quality Assurance Tester

Object of testing: #4026
Tier of Testing: 2
Date: 21/06/2021

Testing Process:

  1. Loaded C&D an setup Settings page like I want it to, calibrated throttles -> Check issues
  2. Checked dispatch page and refueled -> Check issues
  3. Tested all ground services and pushback
  4. Checked ATC page on this VATSIM flight
  5. I do not have a navigraph charts sub
  6. Played with ToD and LDG Calculators inflight

Negatives:

  • PTU toggle in settings page does not move, it just changes color
  • Something was wrong with the Dispatch page. I had no option to load my simbrief FPL. Username was set in the Settings menu. Consequently i also couldnt test OFP page
  • I also undestand the last subpage not being remembered may be a little annoying
  • ATC page does not update, when staying on it. But changing to another page and back to ATC, frequencies will update
  • ATC page didnt give me the TWR, GND and ATIS of LOWW although being within 50nm of the airport on approach

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
Media:
image
image
image
image

Hello. The ATC page is refreshing every two minutes. Wish is also the refresh rate of data coming from vatsim/Ivao.

@frankkopp
Copy link
Member

Tested last commit for the map icons - OK now
image

@St54Kevin
Copy link
Contributor

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.

@ZeroKaa
Copy link
Contributor

ZeroKaa commented Jun 23, 2021

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

@ZeroKaa
Copy link
Contributor

ZeroKaa commented Jun 24, 2021

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.

@St54Kevin . I spot the issue and fixed it. PR still need to be merge
I also increased the font-size

@marcman86
Copy link
Contributor

marcman86 commented Jul 1, 2021

Quality Assurance Tester Report
Discord:marcman86#4907
Object of testing: #4026
Tier of Testing: 1
Date: 1/07/2021

Testing Process:

  • Spawned C&D at EDDP
  • Start up AC and EFB
  • Checked Dashboard page and Simbrief sync
  • Checked Dispatch Overview, OFP and Fuel page
  • Checked Ground page
  • Took off
  • Checked Performance page (TOD and Landing)
  • Checked Navigation & Charts page (QR Code only)
  • Checked VATSIM page (i dont use VATSIM)
  • Checked Settings page
  • Checked Detents page

Negatives:

  • Map plane flickers
  • OFP dont remembers scroll position and zoom level
  • Fuel page plane looks a little pressed (close to the Refuel box), maybe put the fuel time and the plane a litte down
  • Fuel page fontsizes are inconsistent (left outer tank value, right outer tank value) (fig1)
  • Landing page fontsizes are inconsistent/to small (clear, low, medium, manual) (fig2)
  • I dont like the grey hairlines between each line on the setup page

Testing Results: (Not) Passed
Conclusions: Works, just minor fixes
Media:

001
fig1

002
fig2

Hope this helps :-)

hiaaryan and others added 8 commits July 3, 2021 10:38
* 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
@St54Kevin
Copy link
Contributor

St54Kevin commented Jul 12, 2021

Quality Assurance Tester

Object of testing: #4026
Tier of Testing: 2
Date: 12/07/2021

Testing Process:

  1. Full VATSIM EGLL-EDDF flight
  2. Went through all the settings and set everything up to my liking
  3. Loaded simbrief flight plan, refueled via EFB
  4. Tested every ground service
  5. caculated relanding performance
  6. Used ATC tab to tune frequency
  7. EFB pushback did not work
  8. Full flight with monitoring ATC tab and vPilot Controllers
  9. Unfortunately all controllers went offline right after my departure, therefor I wrote this qa airborne (nothing more to test)

Negatives:

  • Sometimes the EFB doesnt take my click when switching pages. I need to click several times to get them to open. But its just from time to time but noticeable
  • After calling the baggage, shutting it off didnt work, button was blue but services were stuck at the plane, the door even stayed open with ENG running(see media), I think this was causing my EFB pushback to not work. Had to use pushback toolbar for both problems. I literally did a full flight this evening with master and it was working fine there
  • The Map didnt have a plane icon to indicate my position

Testing Results: Passed. I couldnt reproduce the ground services bug on another airport

Media:
image
image
image

@St54Kevin
Copy link
Contributor

St54Kevin commented Jul 12, 2021

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

@hiaaryan hiaaryan merged commit cf88fc3 into master Jul 12, 2021
@davidwalschots davidwalschots deleted the flypados-v2 branch November 3, 2021 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet