The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇦🇺Australia acbramley
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇮🇳India mrinalini9 New Delhi
Please ignore #47 merge request.
Here is the patch rerolled for 10.5.x, please review it.
Thanks! - First commit to issue fork.
- First commit to issue fork.
- 🇺🇸United States xjm
I gave @effulgentsia and @hchonov credit on the original issue instead, since there were a lot of other contributions there that had not been credited (major issue triage by the subsystem maintainers, various reviews, etc.).
- 🇺🇸United States xjm
🐛 Move entity preparation from form controller to entity class: "last changed" timestamp not updated and "create new revision" setting not respected when updating entities via Entity API Postponed: needs info was marked as critical by subsystem and core maintainers during a major issue triage for REST, so let's go ahead and promote this issue accordingly.
- @loze opened merge request.
- 🇺🇸United States xjm
Duplicate is probably a better classification.
Saving credits for those who worked on this one.
- 🇺🇸United States xjm
I think duplicate is probably a better classification. :) This was fixed, just in an even better way than we planned.
- 🇺🇸United States xjm
Saving credits for all the contributors to this effort, which partly informed the improved field UI we have today.
Thanks everyone!
- 🇺🇸United States xjm
Sorry, there is a different meta I am confusing this with about the Image to Media upgrade path. This one can probably be called fixed now, rather than outdated:
I'll work on credits
- 🇺🇸United States xjm
I think we're closing outdated these days when really we just want an issue summary update. :)
- 🇺🇸United States xjm
The one issue in the summary about field descriptions was just closed dupe of a newer one.
- 🇺🇸United States xjm
This remains usability technical debt that was descoped and never addressed when we put Media in core, unfortunately.