Skip to content
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

Fix configuration for Sulu 2.0.0 compatibility #42

Merged
merged 10 commits into from
Oct 7, 2019

Conversation

Prokyonn
Copy link
Member

@Prokyonn Prokyonn commented Sep 30, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #issuenum
Related issues/PRs #issuenum
License MIT

What's in this PR?

Update Bundle for Sulu 2.0.0 compatibility

@Prokyonn Prokyonn changed the title Fix toolbar actions for Sulu 2 RC3 compatibility Fix configuration for Sulu 2.0.0 compatibility Oct 4, 2019
@Prokyonn Prokyonn requested a review from niklasnatter October 4, 2019 07:10
@niklasnatter
Copy link
Contributor

can we refactor the controllers to use dependency injection in this PR?

@Prokyonn Prokyonn force-pushed the feature/rc3-compatibility branch from 6485b58 to 9d2f250 Compare October 4, 2019 08:40
@niklasnatter
Copy link
Contributor

you can remove the getJsBundleName method from the admin

@niklasnatter
Copy link
Contributor

can we adjust the routing files to use the yml format in this bundle too? 😇

@niklasnatter
Copy link
Contributor

and i would like to enable php-cs-fixer for this project too. but we should probably do that in a seperate pr 🙂

@niklasnatter niklasnatter merged commit c3ca370 into sulu:develop Oct 7, 2019
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