-
Notifications
You must be signed in to change notification settings - Fork 70
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
Facility address should not be displayed on FE when editor selects "Use the facility's street address" #17514
Comments
By contrast, the VBA facility services on Tugboat currently are showing the Facility Address when this box is checked. See Cleveland VA Regional Benefit Office -home loans. This is not what the Figma design shows, so would want to make sure @thejordanwood signs off on the proposed direction for both these cases. |
Take to UX sync to discuss current behavior in VAMC (not showing address), but VBA is different. |
For reference, here is a previous ticket where the current logic was implemented #7339 |
@davidmpickett I agree with the logic on the ticket you linked.
Additional logic would include: If "use facility street address" box is checked and no wing/floor/room info is added, then nothing should show for this section. |
Thanks @thejordanwood! I am going to update this ticket to turn it in to a ticket to implement the desired behavior |
We have to modify the liquid template a bit. Not a big lift, but worthy of at least 2 points. The conditional needs a bit of restructuring and testing. |
@jilladams @davidmpickett |
Thanks. I updated it's position in the launch plan and the sprint 2 draft plan to reflect this. |
Started to do this but then realized I can't test since data is not available from any mirror of prod yet and the SL tugboat ceased to exist after the merge. |
Description
In Tugboat, the VAMC Top Task pages and VBA Service accordions are displaying the Facility Address when "Use Facility Address" is checked in the Service Location. This is not the desired behavior. VAMC service accordions currently have the correct behavior.
Examples from Tugboat
VAMC Top Task Page - Los Angeles
VBA Facility Service Accordion - Cleveland
VAMC Facility Service Accordion - Pittsburgh
Eng notes
In the liquid template, for the address under the "name of clinic heading": a conditional is missing to check if the "use facility street address" checkbox is checked, before printing.
Acceptance Criteria
The text was updated successfully, but these errors were encountered: