-
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
Add Featured Content ("in the spotlight") to VBA Facility Content Type #15911
Comments
This was listed as a comment on 14699, but doesn't look like that ever got addressed or turned into a ticket |
See also Laura's comments on implementation #13799 (comment) |
Re updated description. To avoid confusion, I would narrow this to only the spotlight piece
Those other three items are handled by entirely different parts of the system. |
Content spec (VBA specific)VBA Facility Content Type
Centralized Content | VBA -- benefit office content
Content spec (Vet center and beyond)Featured Content - paragraph type
Vet Center
|
@omahane let me know if this spec makes sense or if you need any clarifications |
@swirtSJW @davidmpickett Do you have time to look at this, as I've already run into some unexpected trouble. 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. |
After a quick Zoom, Dave noticed that I had reused the field_featured_content rather than made a new field_local_spotlight. Solved. |
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 |
Responding to @omahane comment on the PR back on the main ticket for visibility. How about:
|
@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. |
Made these charts for my own sanity Stuff that's actually part of this ticket
Not part of this ticket, but has confusingly similar names
|
Uploading video clip for async demo: |
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:
Use the Featured Content paragraph from Vet Center
Also see Laura's comment in 13799
-Change label to Spotlight content - this will help with consistency and lessen confusion by aligning CMS and FE naming conventions
-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
The text was updated successfully, but these errors were encountered: