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

Update Service Location paragraph type to account for VBA #14704

Closed
5 of 6 tasks
xiongjaneg opened this issue Aug 8, 2023 · 16 comments
Closed
5 of 6 tasks

Update Service Location paragraph type to account for VBA #14704

xiongjaneg opened this issue Aug 8, 2023 · 16 comments
Assignees
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide

Comments

@xiongjaneg
Copy link
Contributor

xiongjaneg commented Aug 8, 2023

Description

Drupal implementation for VBA Service Location paragraph type is built to match the spec #13128. (Note from Dave - the spec has not been updated to reflect the FE design, so it is wildly out of date)

Collaborate with DaveP if questions arise based on the spec during implementation.

Change management notes

We want to add new VBA fields to Service Location, but do not want those fields to be visible in Prod to VAMC editors yet. That requires Change Management. #14758

Acceptance Criteria

@xiongjaneg xiongjaneg added Needs refining Issue status Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team labels Aug 8, 2023
@jilladams jilladams changed the title Implement updated Service Location paragraph type Create updated Service Location paragraph type Aug 10, 2023
@jilladams
Copy link
Contributor

@xiongjaneg we need to cut a follow up ticket to put in the backlog for exposing these fields and change management, when the time comes.

@xiongjaneg xiongjaneg changed the title Create updated Service Location paragraph type Create VBA Service Location paragraph type Aug 10, 2023
@xiongjaneg xiongjaneg changed the title Create VBA Service Location paragraph type Update Service Location paragraph type to account for VBA Aug 17, 2023
@xiongjaneg xiongjaneg removed the Needs refining Issue status label Aug 24, 2023
@xiongjaneg
Copy link
Contributor Author

xiongjaneg commented Sep 27, 2023

  1. Does this include anything about restrictions? In the spec, there's a cardinality restriction on Facility.
  2. Is 3 functional or does it need to be increased?
  3. Is there work that needs to come before this? Yes. There could be more info added to the spec.

@omahane
Copy link
Contributor

omahane commented Oct 2, 2023

@xiongjaneg I want to confirm that this is the design we're talking about reviewing in in the first AC: https://www.sketch.com/s/891d33ae-152d-471c-ab5d-9aedf89cf6ff/p/FC995695-308A-4493-8461-D9AFE6DC6B25/canvas

@xiongjaneg
Copy link
Contributor Author

@thejordanwood Can you confirm the Sketch link Christian is referring to is the most recent? Thank you!

@thejordanwood
Copy link

@xiongjaneg @omahane This is the Figma file that will have the Service location paragraph type. It's a bit confusing, but CMS designs are in Figma and front end designs are in Sketch.

I just commented on my Service locations ticket and pointed out that, while I've done some UX review of this, I haven't completed the work yet. I'm missing some of the suggestions in Dave's spreadsheet. Specifically the suggestion for better help text to describe what Service Locations are and the addition of the delivery method field.

I'm now realizing that perhaps I should have completed that ticket first, before completing work for Service Regions and other content types that contain the Service Location section.

@xiongjaneg
Copy link
Contributor Author

@omahane Do you have what you need to continue work on this?

@omahane
Copy link
Contributor

omahane commented Oct 6, 2023

@xiongjaneg @thejordanwood @davidmpickett Can I get some clarity on the title? Is this something that an editor is going to manually create/edit?

Other facility service titles are generated:
Screenshot 2023-10-06 at 6 46 32 AM

Patterns for service titles

Vet Center Service title

  • [facility name] - [service name]

VAMC System Health service title

  • [service name] at [system name]

VAMC Facility Health service title

  • [service name] - [facility name]

@omahane
Copy link
Contributor

omahane commented Oct 6, 2023

The other thing I want to confirm is that, aside from title (which I've asked about above), the only net new field is "Service description." Otherwise, it's mostly help text changes/additions. Is that right, @thejordanwood ?

@davidmpickett
Copy link
Contributor

@davidmpickett
Copy link
Contributor

The other thing I want to confirm is that, aside from title (which I've asked about above), the only net new field is "Service description." Otherwise, it's mostly help text changes/additions. Is that right, @thejordanwood ?

No. There are at least 4 net new fields:

  • task-oriented descriptive header
  • Service location description
  • Service modalities
  • appointments ( might need its own nested paragraph type similar to Service location address)

This is what I was referring to when I mentioned the spec was wildly out of date

@davidmpickett
Copy link
Contributor

@xiongjaneg @thejordanwood @davidmpickett Can I get some clarity on the title? Is this something that an editor is going to manually create/edit?

Other facility service titles are generated: Screenshot 2023-10-06 at 6 46 32 AM

Patterns for service titles

Vet Center Service title

  • [facility name] - [service name]

VAMC System Health service title

  • [service name] at [system name]

VAMC Facility Health service title

  • [service name] - [facility name]

This is a good question, but it's not technically part of Service Location, right? It's something that should be addressed on VBA Facility Service?

@omahane
Copy link
Contributor

omahane commented Oct 6, 2023

You're so right, Dave. I was totally conflating the two, having looked at the Figma file from here: #14704 (comment)

@omahane omahane assigned omahane and unassigned swirtSJW Oct 9, 2023
@omahane
Copy link
Contributor

omahane commented Oct 9, 2023

Per Slack conversation with DaveP, we'll be handing the Appointments structure in #14837, rather than doing so here.

@jilladams
Copy link
Contributor

@omahane can you confirm the state of this ticket? Not all ACs are checked, and not sure ifyou need an assist with anything since Jane is gone?

@omahane
Copy link
Contributor

omahane commented Oct 11, 2023

The accessibility review AC is going to have to wait until we've gotten sign-off from @mmiddaugh that our free text fields are the right approach.

@jilladams
Copy link
Contributor

Remaining AC moved to #15649 closing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Drupal engineering CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Regional office CMS managed VBA product owned by the Facilities team sitewide
Projects
None yet
Development

No branches or pull requests

6 participants