-
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
VACMS-16827 Fix loading of alt text field definitions. #16832
Conversation
Discussed with @maortiz-27-80 -- due to Friday-ness and upcoming holiday, Marcia let us know we can go ahead and put this PR up for CMS team review. I added the platform-cms-drupal-engineers team as reviewers, but if a different team should be added as reviewers, please let us know. |
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.
All QA steps passed 👍🏻
Thank you for the fantastic QA steps!
Description
Relates to #16827
Testing done
Screenshots
This is the field screenshot (an image field) on a content model document not throwing a white screen error.
Here is the validation working as intended
QA steps
What needs to be checked to prove this works?
What needs to be checked to prove it didn't break any related things?
What variations of circumstances (users, actions, values) need to be checked?
As as an admin
Definition of Done
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing
Is this PR blocked by another PR?
DO NOT MERGE
Does this PR need review from a Product Owner
Needs PO review
CMS user-facing announcement
Is an announcement needed to let editors know of this change?