symfony-cmf / symfony-cmf-docs

The documentation for the symfony content management framework
http://cmf.symfony.com
44 stars 157 forks source link

(DoctrinePHPCRBundle) Document necessary App document mapping in introduction #852

Closed torreytsui closed 5 years ago

torreytsui commented 5 years ago

As part of a fix of a hard-coded mapping, default application level mapping is no longer offered out of the box (https://github.com/doctrine/DoctrinePHPCRBundle/pull/330).

In a minimal setup, mappings section is now necessary in most cases to offer document mappings on application level.

Since the hard-coded mapping was incorrect in the first place, this document change applied in all maintained versions. This PR updates the documentation to reflect this.

torreytsui commented 5 years ago

Thanks for the feedback! I have updated the PR accordingly. It is now target to master branch. Please can you have another look?