Problem/Motivation
The documentation in the entity_generate plugin is misleading.
Proposed resolution
See patch below.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | migrate_plus-3283457-2-improve_documentation.patch | 2.19 KB | benstallings |
The documentation in the entity_generate plugin is misleading.
See patch below.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | migrate_plus-3283457-2-improve_documentation.patch | 2.19 KB | benstallings |
Comments
Comment #2
benstallings commentedComment #3
danflanagan8This is a really good idea if you ask me! Here are my humble thoughts on the proposed changes.
More likely this example intends for
tagsto be an array of strings that correspond to tag names. I also don't think we need to mention XML, even though it's true that the source might be XML.The new example is super cool, but it doesn't look quite right to me. I would think the following would work better:
I think that would get the alt from field_image correctly onto the media instead of having to use a default value. This uses the image's alt for the media name in order to demonstrate thevalueskey. I think including an example of the default value is a great idea so I put the uid on there.I haven't tested that though. I should do that!I tested and made some edits.