Conversation
@@ -23,38 +23,212 @@ Configuration | |||
|
|||
# app/config/config.yml | |||
cmf_core: | |||
document_manager_name: default | |||
role: IS_AUTHENTICATED_ANONYMOUSLY # used by the publish workflow checker | |||
document_manager_name: null # used for the twig functions to fetch documents |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be manager_name
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed it should, but its not. i'll do a separate PR once this change is merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rmsint: it would be consistent, yes. but unfortunately its like this in
the actual bundle atm. we should clean this up however. care to do a pr?
or open an issue so we don't forget?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I will do a PR on the CoreBundle to change it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not use ~
?
</extension> | ||
</extensions> | ||
</sonata-admin> | ||
</config> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wouterj there was some guessing involved here. is this xml config more or less correct?
<config> | ||
<sonata-admin> | ||
<!-- ... --> | ||
<extensions> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this wrapper element
documentation for publish workflow
This documentation is for the not yet merged publish workflow refactoring.
This is going to conflict with #148 - i suggest i manually port the still relevant parts of #148 over to here once its merged.