department-of-veterans-affairs / va-mobile-app

"If VA were a company, it would have a flagship mobile app."
https://department-of-veterans-affairs.github.io/va-mobile-app/
15 stars 2 forks source link

Bug - Sev 3 - PROD Issue (MOBILE): VVC on VA device display as VVC @ Home vs. VVC on VA Device (BE: Update Appt Type Mapping to acct for Video Connect at VA location) #6542

Closed MekoHong closed 1 year ago

MekoHong commented 1 year ago

Description

An issue was identified that Veterans have missed their appts because they thought their appointment was a VVC-at home appt (as displayed on the appt list on the mobile app). However, the appt should be displayed as Video Connect on VA Device.

Investigation is needed by the Appointments team identify what is needed to address and display the information accurately:

Current Tasks

The appointments FE team has created the below ticket to address Issue: 63865 has been created by the appointments team to modify logic to determine when to show VVC - Home vs. VVC-VA device.

We will need to stay connected with the appointments team to determine what changes Mobile will need to make to be nsync and display the correct modality data

Update: Mapping on the web

See thread for context: https://dsva.slack.com/archives/C018V2JCWRJ/p1692904186776569

Acceptance Criteria

  1. Reflect mapping Update appointAppointment type mapping; should match above web mapping. Note the main change is to check for patientHasMobileGfe & depreciation of MOBILE_GFE

Test Account:

alfred.castillo@id.me

MekoHong commented 1 year ago

Based on convo, it looks like FE already has logic to account for VA_Video_Connect_Onsite appointments. Change will only be needed on the backend and we do not have to add a new version.

image

jperk51 commented 1 year ago

The onsite entry above needs to be added to the mapping in modules/mobile/app/models/mobile/v0/adapters/vaos_v2_appointment.rb line 242 following the rules listed in the ticket description

MekoHong commented 1 year ago

Screenshots from testing; aligned with web and shared update in DSVA thread... https://dsva.slack.com/archives/C018V2JCWRJ/p1693428535174719 image (12) IMG_7826