Skip to content
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

Add extension registration #19

Closed
wants to merge 1 commit into from

Conversation

samwilson
Copy link
Contributor

@samwilson samwilson commented Nov 13, 2017

This change:

  • adds extension.json (with backwards compatible loading from
    Purge.php);
  • fixes a possible conflict with a message name;
  • removes installation instructions etc. from README and points
    instead to the mediawiki.org page; and
  • removes the global variables from the navigation links hook.

Refs: #3

This change:
* adds extension.json (with backwards compatible loading from
  Purge.php);
* fixes a possible conflict with a message name;
* removes installation instructions etc. from README and points
  instead to the mediawiki.org page; and
* removes the global variables from the navigation links hook.

Bug: GH Hutchy68#3
@samwilson
Copy link
Contributor Author

@Hutchy68 do you have a moment to look at this?

Copy link
Contributor

@kizule kizule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok.

@kghbln
Copy link
Contributor

kghbln commented Feb 8, 2018

Before the release of 1.1.0 the license identifier should be updated. Apart from that +1

@varnent
Copy link

varnent commented May 9, 2020

@Hutchy68 - thank you for this extension - any chance some of the pull requests can be merged? :)

@Hutchy68 Hutchy68 mentioned this pull request Jun 11, 2020
@Hutchy68
Copy link
Owner

closing as it was merged after some changes see #26

@Hutchy68 Hutchy68 closed this Jun 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants