department-of-veterans-affairs / va.gov-cms

Editor-centered management for Veteran-centered content.
https://prod.cms.va.gov
GNU General Public License v2.0
99 stars 69 forks source link

[Events] Implement NCA, VBA, and Vet Center facilities as Event locations #10536

Closed jilladams closed 5 months ago

jilladams commented 2 years ago

Status

[2024-05-17] The following facilities should be available in Drupal via the Facilities API and should be able to be part of this implementation. Daniel to update the estimate (next week). [2024-05-07] NCA's exist in nodes in Drupal, so discovery is not needed and the work can commence.

Description

Editors need the ability to choose any VA Facility as an Event Location when they're creating or editing events. The following event locations should now be available in Drupal via the Facilities API and should be made available to editors as events locations:

Note that VAMC was removed from the list because VAMC Facilities are already available as event locations (encapsulates Medical Centers and VA Clinics). We want to ensure as part of this implementation not to regress the existing VAMCs (but there's nothing additional from VAMC content types that needs to be added right now).

~From 8/30 Wes/Jill sync with Dave: National Cemetery Association (m.va.gov) facilities need to migrate full data to Drupal. Currently, the content type exists but only lists operating status & geolocation: https://prod.cms.va.gov/admin/content?title=&type=nca_facility&moderation_state=All&owner=All https://github.com/department-of-veterans-affairs/va.gov-cms/issues/10535 tracks an effort to get NCAs listed as Event locations. In order for that to be possible, we need address data available to display in front-end Event templates. This ticket represents the discovery to work out what's required to flesh out NCA nodes, include address, and enable #10535. Facilities folks have performed / are performing similar work for VBA facilities, so @dsinla and Facilities Drupal folks or Swirt are a good place to start with conversations to understand the work.~

Acceptance Criteria

swirtSJW commented 2 years ago

Basic outline of steps

This feels a little odd that PWS would be picking this up, as it seems like facilities team work even though events is driving the urgency.

dsinla commented 2 years ago

While performing review of tickets in backlog: Discussing with @JayDarnell: Entity reference field on Events does not currently support NCA facility nodes as a possible target

The question is around priority - @dsinla to discuss where this falls in long term roadmap with @davidconlon Are there services at NCA facilities? Do we just expand on data to support events? then come back later for services?

FranECross commented 6 months ago

Description updated to include all facilities migrated into Drupal via the facilities API (NCA, VAMC, VBA, Vet Center, Vet Center Outstation). Another ticket will be created to accommodate any facilities not yet in Drupal to determine how to make them available to the CMS front end for editors to choose. cc @dsasser @jilladams

dsasser commented 5 months ago

@FranECross @jilladams I'm a bit unclear on which 'VAMC' facilities should be allowed. Should we allow any of these that begin with VAMC?

Screenshot 2024-05-20 at 4 42 41 PM

FranECross commented 5 months ago

@dsasser Regarding your question above☝🏼... I see what you mean. Those aren't locations, they're pages/info/etc. Let me get clarification from Dave C, unless @jilladams knows offhand if there's a way to narrow by locations that events might me at?

jilladams commented 5 months ago

Noted in Slack: VAMC Facilities (which covers medical centers and VA clinics) are already exposed as locations that can be chosen on Events. So I think the note here is just not to regress that, but there's nothing additional from VAMC content types that needs to be added right now. (Or prob ever.)

FranECross commented 5 months ago

Thanks, @jilladams ! I tweaked the Description a bit, and also added AC to ensure the existing VAMC Medical Centers and VA Clinics are still available.

jilladams commented 5 months ago

@dsasser BEFORE YOU MERGE: I think per Dave, the vision here is that a VAMC or Outreach editor should be able to create an Event using any Facility as the location. But: I would like to test a couple things (or: make sure that a couple of things are tested) in a Tugboat for my own state of mind, before we ship:

dsasser commented 5 months ago

@jilladams

As a Vet Center or VBA editor...Am I able to create Events?

No. Only the 'Content creator - Outreach Hub', 'Content creator - VAMC', and 'Content admin' roles can create Events, but other roles can edit them after the fact providing the Events fall within their assigned Section.

A Tugboat is available here for testing.

jilladams commented 5 months ago

Verified:

I requested content release to build the FE & verify, but: it'll just be a reg VAMC / Outreach event, and location won't affect crumbs.

I am reassured. I'll take a look at the FE tomorrow to make sure, but when your PR is ready, should be ok to merge once that's out of the way.

jilladams commented 5 months ago

No surprises, clear to merge. Screenshot 2024-06-04 at 9 19 39 AM

jilladams commented 5 months ago

Verified in Prod with Vet Center, VBA Regional Office, Cemetery:

Screenshot 2024-06-11 at 3 08 59 PM Screenshot 2024-06-11 at 3 08 51 PM Screenshot 2024-06-11 at 3 08 40 PM