-
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
Last updated by human not set when Vet Center Service is saved/updated. #15124
Comments
Depending on changes, the new "Vet Center content review" KB article which was written to support Vet Center editors in reviewing/updating their content (currently in draft) may need to be updated. |
This could be resolved by removing inline entity field |
moved larger IEF removal to #17374 |
@xiongjaneg I have been working on this and the #15125 most of the day, and neither changes are able to made with config, as Michael and I expected. As a result, to try to do this will take custom coding that I don't think we want to do if we're intending to remove IEF from Vet Centers anyway. Can we discuss another Drupal priority ticket? |
@omahane Absolutely makes sense. I'll move both of those tickets back to backlog. I've moved #17191 address block work to high priority. cc @mmiddaugh @jilladams |
IEF removal first ticket is #17374 |
This defect will be resolved once the IEF changes are implemented. Leaving this defect in backlog to confirm and close after #18003 |
Can be validated / closed when #18003 ships |
Describe the defect
When an editor edits a Vet Center Service, they have to do so using the Inline Entity Form on the Vet Center node. This does not update the last saved by an editor field on the Vet Center Service node
![image](https://private-user-images.githubusercontent.com/5752113/266133468-1672c74e-0a75-4abc-909f-427c06372b52.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxNDc0MDEsIm5iZiI6MTczOTE0NzEwMSwicGF0aCI6Ii81NzUyMTEzLzI2NjEzMzQ2OC0xNjcyYzc0ZS0wYTc1LTRhYmMtOTA5Zi00MjdjMDYzNzJiNTIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTBUMDAyNTAxWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZTgzYTcyNzczMjI1YmViZDM0YzczZjdhNmY4NTQ2YzJiMDdlNjljMDY3Nzc4OTQ3N2ExZjQ1NWMzYTRiNzA4YyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.FIC5qn5tvQESNqnBpb9gW-GuQ-UoJCNA6W2Wp8p9EKA)
To Reproduce
Steps to reproduce the behavior:
AC / Expected behavior
The text was updated successfully, but these errors were encountered: