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

Add Featured Content ("in the spotlight") to VBA Facility Content Type #15911

Closed
8 tasks done
davidmpickett opened this issue Oct 30, 2023 · 18 comments
Closed
8 tasks done
Assignees
Labels
Content model [CMS feature] The bones of the CMS Content CMS team practice area 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 Vet Center CMS managed product owned by Facilities team

Comments

@davidmpickett
Copy link
Contributor

davidmpickett commented Oct 30, 2023

User story

As a VBA editor, I want to be able to feature a timely event or other information to get Veterans' attention.

Background

Summary of business feedback/decisions from Office of Strategic engagement:
Design should include:

  • 1 centralized/national spotlight with 2 local/editor level spotlights

Use the Featured Content paragraph from Vet Center

Also see Laura's comment in 13799

  • Featured content:
    -Change label to Spotlight content - this will help with consistency and lessen confusion by aligning CMS and FE naming conventions
  • Section Header
    -Change label to Title - section is a term used very specifically for something else within the CMS so should be avoided whenever possible to refer to other things.
    -Keep field as required
    -Keep field as a text input
    -Keep character limit at 80

Acceptance criteria

  • @davidmpickett drafts a clear spec for fields that need to change and reviews with @omahane before @omahane starts implementation
  • @davidmpickett assesses impact of changes on Vet Centers, and brings that to Product for approval / change management. If Laura's requested updates to fields used on Vet Centers becomes a blocker, that work should split off from adding the fields AS IS to VBA
  • VBA Featured Content is implemented in the same way as Vet Center Featured Content (with the following changes)
    • Featured content label is changed to Spotlight content
    • Section Header label is changed to title
  • There is one centralized content field and one local content field with cardinality of two
  • @davidmpickett QAs @omahane's PR
  • @laflannery a11y review
@davidmpickett davidmpickett added 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 Oct 30, 2023
@davidmpickett
Copy link
Contributor Author

This was listed as a comment on 14699, but doesn't look like that ever got addressed or turned into a ticket

@xiongjaneg xiongjaneg added the Needs refining Issue status label Oct 30, 2023
@davidmpickett
Copy link
Contributor Author

See also Laura's comments on implementation #13799 (comment)

@xiongjaneg
Copy link
Contributor

@davidmpickett
Copy link
Contributor Author

Re updated description. To avoid confusion, I would narrow this to only the spotlight piece

  • Gov Delivery
  • national social media only (no local)
  • 1 centralized/national spotlight with 2 local/editor level spotlights
  • AVA and VERA

Those other three items are handled by entirely different parts of the system.

@davidmpickett davidmpickett added Content model [CMS feature] The bones of the CMS Content CMS team practice area Vet Center CMS managed product owned by Facilities team labels Nov 2, 2023
@xiongjaneg xiongjaneg removed the Needs refining Issue status label Nov 2, 2023
@davidmpickett
Copy link
Contributor Author

davidmpickett commented Nov 9, 2023

Content spec (VBA specific)

VBA Facility Content Type

  • Remove field_cc_national_spotlight_2
  • Remove field_cc_national_spotlight_3
  • Add field_local_spotlight
    • Field type: Entity reference revision -> "Featured content" paragraph type
    • Label: Spotlight content
    • Cardinality: 2
    • Translatable: No
  • Add fieldset "Spotlight content" to Form Display
    • Description: Highlight up to two benefit office activities, such as events or programs.
    • Fields to include in fieldset
      • National spotlight tooltip
      • National spotlight 1
      • field_local_spotlight

Centralized Content | VBA -- benefit office content

  • Remove Featured content "Do you have the latest VA form?"
  • Remove Featured content "Six essential facts"

Content spec (Vet center and beyond)

Featured Content - paragraph type

  • Change label -> 'Spotlight content'
  • Machine name will stay: featured_content
  • field_section_header
    • Change label -> 'Title'

Vet Center

  • field_vet_center_feature_content
    • Change label -> 'Spotlight content'
    • Do these Paragraphs (stable) field widget settings actually do what they say they do? If so, update them.
      • Paragraph Title -> spotlight content
      • Plural Paragraph Title -> spotlights
  • "Featured content" fieldset
    • Field group label -> Spotlight content
  • "National featured content" fieldset
    • Field group label -> National spotlight content
    • Tooltip Description -> Why can’t I edit this? This national feature spotlight managed by VHA will display alongside locally-relevant content.

@davidmpickett
Copy link
Contributor Author

@omahane let me know if this spec makes sense or if you need any clarifications

@mmiddaugh
Copy link
Contributor

mmiddaugh commented Nov 15, 2023

if we're moving to "Spotlight", the Vet Center dashboard will also need to be updated.

dashboard

@omahane
Copy link
Contributor

omahane commented Nov 15, 2023

@swirtSJW @davidmpickett Do you have time to look at this, as I've already run into some unexpected trouble.

Screenshot 2023-11-15 at 2 04 56 PM

I'm thinking that this means that other content types that are using the paragraph already have more than the number I'm trying to limit it to, which means that the limit is not set per content type but on the paragraph itself, which, as a shared field means that we are trying to butt in on someone else's content.

So, @dave Pickett I think we might be already running into a change management issue with #1. We could always keep it unlimited and say that only the first two will ever show, but that's clunky. Or we make a new Paragraph. I'll bring this discussion to the issue.

@omahane
Copy link
Contributor

omahane commented Nov 15, 2023

@swirtSJW @davidmpickett Do you have time to look at this, as I've already run into some unexpected trouble.

After a quick Zoom, Dave noticed that I had reused the field_featured_content rather than made a new field_local_spotlight.

Solved.

@davidmpickett
Copy link
Contributor Author

Fun and not-at-all-confusing fact:
The "Featured content" field on VAMC Detail Pages and Benefits Detail Page uses the Rich Text and Page-Specific Q&A paragraph types rather than the Featured Content (soon to be Spotlight Content) paragraph type.

Screenshot 2023-11-15 143337

@laflannery
Copy link
Contributor

I want to note that the screenshot above is Featured Content, not Spotlight content for VAMCs. It is separate from what is being worked on in this ticket. The spotlight content is on the VAMC System page and is quite different from the pattern being worked on here and is documented in this issue to be handled at future time

@davidmpickett
Copy link
Contributor Author

davidmpickett commented Nov 16, 2023

Responding to @omahane comment on the PR back on the main ticket for visibility.

How about:

  • "Spotlight content" - for fieldset that groups everything (could also be "In the spotlight")
  • "National spotlight" - for the cc fetch fields
  • "Local spotlight(s)" - for the paragraph type heading
  • "spotlight" - for the add another button (could also be "spotlight content")
  • Let's move the help text about "add up two office activities" down closer to where the user would actually perform that action

spotlights

@davidmpickett
Copy link
Contributor Author

I want to note that the screenshot above is Featured Content, not Spotlight content for VAMCs. It is separate from what is being worked on in this ticket. The spotlight content is on the VAMC System page and is quite different from the pattern being worked on here and is documented in this issue to be handled at future time

@laflannery Sorry for any confusion. I was documenting it because this is the very thing that tripped up @omahane earlier today. He was accidentally reusing the field_featured_content, despite Featured Content being a totally separate thing from Spotlight Content.

@davidmpickett
Copy link
Contributor Author

davidmpickett commented Nov 16, 2023

Made these charts for my own sanity

Stuff that's actually part of this ticket

Thing Current Label New Label Machine name
Paragraph type that powers "In the spotlight" on VBA and Vet Center Featured Content Spotlight Content featured_content
Entity Reference Field that powers the above paragraph type for Vet Centers Featured content Local spotlight(s) field_vet_center_feature_content
Entity Reference Field that powers the above paragraph type for VBA Spotlight content Local spotlight(s) field_local_spotlight
Label on the button for adding another instance of paragraph type Add Spotlight content Add Spotlight ? I am unclear on what actually determines this label

Not part of this ticket, but has confusingly similar names

Thing Label Machine name
Paragraph type that powers "In the spotlight" on VAMC List of link teasers list_of_link_teasers
Entity Reference Field that powers Featured Content for VAMC and Benefit Detail Pages Featured content field_featured_content
Paragraph types that power Featured Content for VAMC and Benefit Detail Pages Rich text & Page-Specific Q&A wysiwyg & q_a

@davidmpickett davidmpickett removed their assignment Nov 17, 2023
@swirtSJW
Copy link
Contributor

image
This location seems misleading.

@swirtSJW
Copy link
Contributor

I fixed the location.
image

@swirtSJW
Copy link
Contributor

This has been merged #16145
Nice work @omahane

@davidmpickett
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content model [CMS feature] The bones of the CMS Content CMS team practice area 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 Vet Center CMS managed product owned by Facilities team
Projects
None yet
Development

No branches or pull requests

7 participants