Skip to content

[ContentBundle] Prefix contentbundle template path #524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 10, 2014

Conversation

dantleech
Copy link
Contributor

This PR automatically adds the internal content bundle templates directory to the sulu_core configuration.

This removes the need to define structures.content.paths.xxx.path: %kernel.root_dir%/vendor/foo/bar/bar/bar/Content/templates

@dantleech dantleech force-pushed the enhancement/prefix-content-bundle-template-path branch from a94736d to 66c704d Compare November 7, 2014 15:35
@dantleech
Copy link
Contributor Author

@wachterjohannes I think you opened an issue about this?

@wachterjohannes
Copy link
Member

@dantleech i dont think i have opened a ticket for that ... but this is a good idea to handle it in a Compiler Pass!

@wachterjohannes
Copy link
Member

@dantleech PR is ok can you merge it with develop?

@dantleech
Copy link
Contributor Author

@wachterjohannes its not handled in a compiler pass, but uses the bundle configuration prefix feature.

will rebase and merge :)

@dantleech dantleech force-pushed the enhancement/prefix-content-bundle-template-path branch from 66c704d to 1b8bf93 Compare November 10, 2014 10:18
dantleech added a commit that referenced this pull request Nov 10, 2014
…le-template-path

[ContentBundle] Prefix contentbundle template path
@dantleech dantleech merged commit 787eb66 into develop Nov 10, 2014
@dantleech dantleech deleted the enhancement/prefix-content-bundle-template-path branch November 10, 2014 10:19
dantleech added a commit to sulu/sulu-standard that referenced this pull request Nov 11, 2014
dantleech added a commit to sulu/sulu-standard that referenced this pull request Nov 11, 2014
wachterjohannes added a commit to sulu/sulu-standard that referenced this pull request Nov 11, 2014
alexander-schranz pushed a commit to alexander-schranz/sulu that referenced this pull request Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants