Feedback on page: /guides/v2.2/ext-best-practices/tutorials/copy-fieldsets.html

[WebKenth]

General issue

Description:

It is not declared where the copy class should be placed.

A slimmed down version of a class without any hint of where to put it is really bad documentation.

Possible solutions:

Rather than using the magic 3 dots to hide important information, show us the entire class.
The namespace alone would reveal key information on where to place it.

This hardly qualifies as an example of how to copy fieldsets.

Be explicit, tell us where to place the extension_attributes.xml, don't assume we know that it should be placed in the etc folder.

[michizhou]

I can take this issue. If you could provide more supporting docs that would be much appreciated! #SQUASHTOBERFEST

[WebKenth]

@michizhou couldn't get it to work at all ended up using a plugin listening on beforePlace on the \Magento\Sales\Api\OrderManagementInterface and added my data from the Quote

All i needed was to copy a field when converting from Quote to Order and this doc seems like the right way to do it, but alas i couldn't get it to work using fieldsets.

[michizhou]

@WebKenth In that case, are there other related tasks to which I could contribute? I would appreciate being able to help resolve this issue in any way I can!

[shrielenee]

@michizhou @WebKenth Hello all - what is to be done on this issue? What is the status? Thanks!

[michizhou]

@shrielenee I am not too sure. Ask @WebKenth for a more in-depth status update.

[lorikrell]

THis was completed per PR #3992. Closing issue.