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

[MultiDomainBundle] Add MultiDomainBundle #633

Merged
merged 1 commit into from
Aug 18, 2015

Conversation

wimvds
Copy link
Contributor

@wimvds wimvds commented Aug 14, 2015

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

This PR contains some small bugfixes (ie. for search & adminlists) and includes the MultiDomainBundle that adds multi-domain & multi-site support.

@roderik roderik changed the title Add MultiDomainBundle [MultiDomainBundle] Add MultiDomainBundle Aug 14, 2015
@krispypen krispypen added this to the 3.3.0 milestone Aug 14, 2015

use Symfony\Component\Security\Core\SecurityContextInterface;

class DefaultAdminPanelAdaptor implements AdminPanelAdaptorInterface
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe you should split this up in 3 different adaptors?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That would make overriding the default behavior a lot harder... Right now you just have to override kunstmaan_admin.admin_panel.adaptor with your own custom implementation to change default behavior (you can inherit the current behavior from it as all methods are protected or public).

@wimvds wimvds force-pushed the feature/multi-domain-bundle branch 3 times, most recently from be3c819 to b7a8c81 Compare August 17, 2015 13:31
@krispypen
Copy link
Contributor

@roderik 👍

@wimvds wimvds force-pushed the feature/multi-domain-bundle branch from 61fdf62 to 0057170 Compare August 18, 2015 12:07
fix visibility of slug router properties

multi domain routing

refactor language switcher

fix list actions in adminlist template

Allow creating new homepages on pages admin list

multi domain seems to work on front-end

limit backend pages to current host

fix templates & link chooser

add host switch in backend

fix for site switcher

fix sitemap bundle

fix sitemap to include homepage

fix tree for multi site

only display add homepage if user is super admin

fix unique slug generator for multi site

fix search for multi site

fix search page

code cleanup

small fixes

cookbook article & site switch as dropdown

adapt menu bundle for multi site

add flash message in admin area when host override is active

fix routing & code formatting

make security context protected

bugfix page generator + fix code formatting

fix page & pagepart generators

bugfix for previews of unpublished pages
@wimvds wimvds force-pushed the feature/multi-domain-bundle branch from 0057170 to 28e86ea Compare August 18, 2015 13:06
krispypen added a commit that referenced this pull request Aug 18, 2015
[MultiDomainBundle] Add MultiDomainBundle
@krispypen krispypen merged commit d609ab7 into Kunstmaan:master Aug 18, 2015
@yoshz
Copy link

yoshz commented Sep 3, 2015

.. created a new issue for this comment: #684

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

3 participants