symfony-cmf / media-bundle

UNMAINTAINED - Minimalistic interfaces to handle media in the context of the CMF
http://cmf.symfony.com/
30 stars 40 forks source link

Update to latest bundle standards #41

Closed rmsint closed 11 years ago

rmsint commented 11 years ago

To be merged together with https://github.com/symfony-cmf/CreateBundle/pull/70

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? yes
Fixed tickets #29
License MIT
Doc PR https://github.com/symfony-cmf/symfony-cmf-docs/pull/191

http://symfony.com/doc/master/cmf/contributing/bundles.html

TODO sync changed configuration with:

rmsint commented 11 years ago

@lsmith77 do we want to create a related PR for the CreateBundle to update it to the latest bundle standards and keep that PR in sync with this one? We will have to merge them then at the same time.

lsmith77 commented 11 years ago

if you have time to wrap it up by tomorrow then yes ..

rmsint commented 11 years ago

Ok I will do that this evening.

rmsint commented 11 years ago

@dantleech @lsmith77 what's the bundle standard for persistence specific configuration files? Is imagine-persistence-phpcr.xml and serializer-persistence-phpcr.xml OK?

lsmith77 commented 11 years ago

argh .. just noticed that the tests seem to fail for 2.2

rmsint commented 11 years ago

will check

rmsint commented 11 years ago

Found it, see https://github.com/symfony-cmf/MediaBundle/pull/43.