-
Notifications
You must be signed in to change notification settings - Fork 11
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
Extract core logic #134
Comments
Makes sense. Any candidates for logic to be extracted aside from that? On Sat, Aug 27, 2016, 12:32 AM Kiko Beats [email protected] wrote:
|
google analytics should be a docpress plugin or docpress-core |
Hmmm. Google Analytics needs to inject a JS snippet somewhere. Two ways I can think about how this can go:
Indeed, the |
I think that the first point will be a good approach for make easy create other plugins, putting the share content inside meta |
I think that is best to do with injections. Like: file.content = file.content.replace('', '...') On Mon, Aug 29, 2016, 4:47 PM Kiko Beats [email protected] wrote:
|
Currently in this repository have logic very closed with docpress. For example, the logic related about determinate the JS/CSS files to load need to moved in core in favour to make more easy create different docpress themes.
The text was updated successfully, but these errors were encountered: