Skip to content
This repository has been archived by the owner on Dec 18, 2020. It is now read-only.

added translation and configuration #592

Merged
merged 1 commit into from
Jan 29, 2016
Merged

Conversation

danrot
Copy link
Contributor

@danrot danrot commented Jan 19, 2016

informations:

q a
Fixed tickets none
Related PRs sulu/sulu#1233
BC breaks none
Documentation PR none

@@ -21,9 +21,10 @@ public function registerBundles()
new Symfony\Bundle\MonologBundle\MonologBundle(),
new Symfony\Bundle\SwiftmailerBundle\SwiftmailerBundle(),
new Symfony\Bundle\AsseticBundle\AsseticBundle(),
new Sensio\Bundle\FrameworkExtraBundle\SensioFrameworkExtraBundle(),
new Sulu\Bundle\CoreBundle\SuluCoreBundle(),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had To move that one higher, in order to use the %sulu.cache_dir% container parameter in the DoctrineCacheBundle configuration. Would say it is a good idea anyway, since the SuluCoreBundle should initialize the most important stuff and make that available as soon as possible. /cc @chirimoya @wachterjohannes

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

collaboration:
enabled: true
entity_cache: doctrine_cache.providers.sulu_collaboration_entity
connection_cache: doctrine_cache.providers.sulu_collaboration_connection
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are this service ids build by DoctrineCacheBundle seems to be very magic (: we should really think about including StashPHP

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why magic? That's pretty clear to me. But still StashPHP might be a good idea.

wachterjohannes added a commit that referenced this pull request Jan 29, 2016
added translation and configuration
@wachterjohannes wachterjohannes merged commit 3f758c9 into develop Jan 29, 2016
@wachterjohannes wachterjohannes deleted the feature/collaboration branch January 29, 2016 13:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants