-
Notifications
You must be signed in to change notification settings - Fork 27
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
Allow adding core and additional Mediawiki skins. #997
Conversation
Committed a couple of more mods - commented out non-Vector skins and created a MezaLocalSkins.yml.j2 template to fix Travis CI build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"conf-public" found on line 11 (comment) of config/core/MezaCoreSkins.yml should be changed to "conf-meza". Otherwise, this looks good from here. Haven't tested, but I like it +1:
@djflux thanks for the PR. @freephile thanks for starting this work and for taking a look at the code. I'll test later today hopefully. |
Test failed, probably an intermittent failure. Restarting the failed job. |
Thanks for the PR! |
This pull request is an update of PR #930. It splits out meza core skins into a
MezaCoreSkins.yml
file. It also provides the ability for an administrator to add additional MediaWiki skins via aMezaLocalSkins.yml
file. All of this work was original written by @freephile. Thanks for the great work.Changes
/opt/meza/config/core/MezaCoreSkins.yml
/opt/conf-meza/public/MezaLocalSkins.yml
Issues
Post-merge actions
Post-merge, the following actions need to be addressed: