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

[KunstmaanAdminBundle] Refactor DomainConfiguration #726

Merged
merged 1 commit into from
Sep 24, 2015

Conversation

wimvds
Copy link
Contributor

@wimvds wimvds commented Sep 23, 2015

Q A
Bug fix? yes
New feature? no
BC breaks? yes
Deprecations? no
Fixed tickets

This PR fixes projects that don't use the KunstmaanNodeBundle, so everything works for them as it should again. The DomainConfiguration had to be moved to KunstmaanAdminBundle, so this is a breaking change (it will only impact users that already used the domain configuration in their own projects though).

Upgrade instructions :

  • replace all occurences of kunstmaan_node.domain_configuration with kunstmaan_admin.domain_configuration
  • replace all references to Kunstmaan\NodeBundle\Helper\DomainConfigurationInterface with Kunstmaan\AdminBundle\Helper\DomainConfigurationInterface

finish refactoring

fix documentation

fix form types
@roderik roderik changed the title refactor DomainConfiguration [KunstmaanAdminBundle] Refactor DomainConfiguration Sep 24, 2015
@roderik roderik added this to the 3.3.3 milestone Sep 24, 2015
roderik pushed a commit that referenced this pull request Sep 24, 2015
[KunstmaanAdminBundle] Refactor DomainConfiguration
@roderik roderik merged commit af6aef1 into Kunstmaan:master Sep 24, 2015
@roderik
Copy link
Contributor

roderik commented Oct 4, 2015

@wimvds I want to tag 3.4.0 but I need a UPGRADE-3.4.md file with the change to go from 3.3 to 3.4 in regards to this pull request.

@wimvds wimvds deleted the fix-domain-configuration branch October 7, 2015 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants