-
Notifications
You must be signed in to change notification settings - Fork 71
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
VACMS-12933: Exclude forms from appearing in VACO and VBA results #20391
VACMS-12933: Exclude forms from appearing in VACO and VBA results #20391
Conversation
… to be excluded from views result.
…nomy term View argument query.
Checking composer.lock changes... |
…VBA-and-VACO-sections
Checking composer.lock changes... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great and works well. Very cool patch.
Cypress Accessibility Violations
|
Description
The section taxonomy view uses Taxonomy Entity Index's contextual argument to display content that references the "section" Vocabulary. This works for most content that only references the vocabulary using one field, but VA Forms uses two fields. This results in VA Forms appearing in VBA and VACO sections when they shouldn't be.
From the original issue:
VA Forms have two "administration" fields:
"Form Administration"
a. Machine name: field_va_form_administration
b. Field type: entity_reference.
c. Referencing "Section" taxonomy terms (parents only, using entity reference selection View administration_section)
"Section"
a. Machine name: field_administration
b. Field type: entity_reference
c. Referencing "Section" taxonomy terms. Statically set to term id 196 (OCTO -> Sitewide content team -> Forms) for all forms
This PR removes field_va_form_administration from consideration in the main taxonomy term View.
Relates to #12933
Testing done
Tested in browser using the new argument option.
Screenshots
QA steps
Validate VA Forms are not the wrong section
As as QA Content Publisher
Validate VA Forms are in the correct section
Definition of Done
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing