Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Console Modules/Extensions #703

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

crwgregory
Copy link
Contributor

This request is to allow any module with a type of 'console' to be loaded into the application before the 'console.init' event is called, but also allows for future development by adding a getType function in the module manager that returns any registered modules of a specific type.

Currently modules don't get loaded before the 'console.init' event is called and this is limiting if you want to build a pagekit extension/module that extends the pagekit console.

…red modules by type. Getting all types of modules by 'console' in the Console Application and loading before calling 'console.init'.
@crwgregory
Copy link
Contributor Author

An example of what kind of extension this feature would allow for:
https://github.com/drunken-kungfu/pagekit-template-generator

@gholleyd
Copy link

gholleyd commented Aug 2, 2016

I would like this to merged in. Something like this could be used to create a custom command running on a cron job, which could have several advantages for clearing cache, generating reports, post processing, and a bunch of other things.

@malte-christian
Copy link
Contributor

Indeed, this can be quite useful. But it has to be secured that no extension break the console app, because the console app should also serve as a "rescue tool", if some fatal errors preventing the usage in the browser.

@crwgregory
Copy link
Contributor Author

@MalteScharenberg How do you imagine something like that working? Could you give an example of a situation where you would need to use the console as a "rescue tool" and I can try to implement a safety mechanism into the pull request.

@ahoiroman
Copy link
Contributor

Need this feature urgently. Maybe it would be an option to start ./pagekit --safe which does not include any extensions at all - this could be used to recover a pagekit-instance in case of emergency.

@tobbexiv tobbexiv mentioned this pull request Mar 27, 2017
@ahoiroman
Copy link
Contributor

Any news on that?

@crwgregory
Copy link
Contributor Author

I haven't worked with pagekit for over a year now. If you are wanting to implement this feature feel free to get into the pull request and add in a safety feature. Shouldn't be to hard.

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

Successfully merging this pull request may close these issues.

None yet

4 participants