Page MenuHomePhabricator

Make Vector skin installed by default in Canasta
Closed, ResolvedPublic

Description

Canasta is a MediaWiki-based Docker image:

https://github.com/WikiWorks/Canasta

Its Dockerfile does a variety of things, one of which is to add the line "#cfLoadSkin()" to LocalSettings.php for every skin that is included in Canasta:

https://github.com/WikiWorks/Canasta/blob/master/Dockerfile

In other words, it adds commented-out lines, so that the admin can include them if they want to. This is fine - but one skin, Vector, should be included by default, i.e., not start with "#".

This is a Google-Summer-of-Code 2022 microtask for T301893.

Event Timeline

I did work on this task and works in my environment. Had to change the docker-compose to use my personal image to test this. I used
sed to uncomment the line. Here is my work. Please do give your suggestion for me to improve on.

@Amalpaultech - sorry for the delay. That's a clever solution! Could you just add a comment above that line, explaining what it's doing?

@Yaron_Koren It's fine. I will get the comment added, and thanks for your suggestion.
EDIT: Here is the updated work

Okay, this looks pretty good. Could you create a pull request for this change?

srishakatux subscribed.

This task seems to have been resolved as per Github. Closing for now.