Skip to content

Enable use on all Wikipedias#34

Merged
Niharika29 merged 1 commit intowikimedia:masterfrom
samwilson:multiwiki2
Nov 7, 2016
Merged

Enable use on all Wikipedias#34
Niharika29 merged 1 commit intowikimedia:masterfrom
samwilson:multiwiki2

Conversation

@samwilson
Copy link
Member

This change updates CopyPatrol to work on all language
Wikipedias that are supported by EranBot. Primarily it
focusses on getting things working with French (for
which translations are encluded here) but with the
exception of the WikiProject prefix cleanup in index.js
there's now nothing here specific to any one language.

Upgrading to this will require a couple of changes in
the .env file, and running composer install.

Bug: https://phabricator.wikimedia.org/T145436

This PR replaces #32 and squashes everything into one commit.

@kaldari
Copy link
Collaborator

kaldari commented Nov 3, 2016

Hmm, the number formatting doesn't seem to be working in French. Currently, I see: "Compteur d'édition : 22,594", but I assume that should be "Compteur d'édition : 22 594". Also "4,055 mots".

@samwilson
Copy link
Member Author

Oops! It was because the Twig number formatting was being configured in route middleware that was never being loaded. I've moved it into App::configureView().

This change updates CopyPatrol to work on all language
Wikipedias that are supported by EranBot. Primarily it
focusses on getting things working with French (for
which translations are encluded here) but with the
exception of the WikiProject prefix cleanup in index.js
there's now nothing here specific to any one language.

Upgrading to this will require a couple of changes in
the .env file, and running `composer install`.

Bug: https://phabricator.wikimedia.org/T145436
@samwilson
Copy link
Member Author

Numbers display correctly now: http://tools.wmflabs.org/plagiabot/en/leaderboard?uselang=fr

@Niharika29
Copy link
Collaborator

This looks great and works nicely. I'm going to merge this but let's not deploy it until we have Eranbot active on frwiki and musikbot configured all fine.

@Niharika29 Niharika29 merged commit f77c316 into wikimedia:master Nov 7, 2016
@samwilson
Copy link
Member Author

Sounds good. Thanks!

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants