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

Move admin to separate package #52

Open
derekisbusy opened this issue Nov 17, 2015 · 1 comment
Open

Move admin to separate package #52

derekisbusy opened this issue Nov 17, 2015 · 1 comment

Comments

@derekisbusy
Copy link
Contributor

Do you want to move the admin controller to a separate package?

There are a couple of advantages to this:

  • admin controls can be left uninstalled and possibly moved to another server and use API controls. This way you could have one server admin other servers using single install of admin package and an API. Also there wouldn't be any admin controls that a person could login to using brute force.
  • we can attach the admin package as a sub module so it can be defined in backend only for advanced app layouts.

So if you are using the base app you would define the admin in your config as a module or sub module.

'modules' => [
    'user' => [
        'class' => 'comyii\user\Module',
        'modules' => [
            'admin' => [
                'class' => 'comyii\admin\Module',
            ]
        ]
    ]
]

And in advanced apps it would only be defined in backend configs.

@kartik-v
Copy link
Member

It is a good enhancement - but probably this can be taken up at the end or a bit later once we know what all functionality can be taken into admin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants