Skip to content
This repository has been archived by the owner on Jul 30, 2022. It is now read-only.

1.39 #116

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

1.39 #116

wants to merge 2 commits into from

Conversation

jdlrobson
Copy link

This fixes warnings in 1.39. There is no urgency with this one. #115 is more urgent given the skin is using incompatible code.

PR Submission:

  • Have tested this PR and it adds a feature or fixes an issue noted below.

Additional Information Below

  • If you remove the line Wikimedia\suppressWarnings(); you'll see various warnings about future breakages. This changes takes care of all of these.

No urgency to merge this one. Those warnings won't likely become a problem until at least MediaWiki 1.41.

isLogged in was deprecated in 1.36 and removed recently.
See T270450 for more information
This makes Pivot skin work again.

Fixes: Hutchy68#114
In 1.36 the skin registration process was modified to support
registration for common skin meta data

This removes warnings in >= 1.39.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant