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
97 stars 69 forks source link

Drupal: Add Banner fields to Vet Center content type #18924

Open davidmpickett opened 1 month ago

davidmpickett commented 1 month ago

User Story or Problem Statement

AS A Vet Center editor I NEED be able to add a banner alert to my Vet Center Facility SO THAT I can communicate urgent information to visitors (e.g. closed for hurricane)

Description or Additional Context

This will follow the exact same model as VBA. The same set of fields that were added to the VBA Facility will be added to the Vet Center Facility. The PR for the prior changes as guidance is https://github.com/department-of-veterans-affairs/va.gov-cms/pull/16076

Possible opportunity to harden the repeated set of fields into a Custom Field https://github.com/department-of-veterans-affairs/va.gov-cms/issues/18929

Screenshot 2024-08-13 at 2 28 39 PM

Steps for Implementation

### New Fields to add
- [ ] Display a banner alert on this facility - field_show_banner - Boolean
- [ ] Banner alert headline - field_banner_title - Text (formatted, long)
- [ ] Banner alert message - field_banner_content - Text (formatted, long)
- [ ] Banner alert type - field_alert_type - List (text)
- [ ] Dismissible by user? - field_dismissible_option - List (text)

Acceptance Criteria

omahane commented 2 days ago

https://github.com/department-of-veterans-affairs/va.gov-cms/pull/16076 is the VBA work that could be a model.

mmiddaugh commented 1 day ago

@Agile6MSkinner we should consider timing - we need to get the business logic nailed down for when it can/should be used and communicate that with editors before this brand new and prominent field is exposed to editors.

Agile6MSkinner commented 1 day ago

@mmiddaugh Definitely. This Drupal story will not expose the banner on the front end, but could confuse editors when the see the fields. I'll get together a CM ticket.