Page MenuHomePhabricator

Improve boundaries between the WikimediaApiPortal skin and WikimediaApiPortalOAuth extension
Closed, ResolvedPublic

Description

Initially, the WikimediaApiPortal skin included hardcoded links to the Special:AppManagement page created by the WikimediaApiPortalOAuth skin. This created a dependency of the skin upon the extension. However, it may be useful to employ the skin on another Wikimedia wiki that does not use the extension in the future. Similarly, it may be desirable to some users to use the extension with a different skin. The goal of this task is to remove the dependence of the skin upon the extension and to identify any other dependencies between them.

Event Timeline

apaskulin created this task.Oct 6 2020, 3:06 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 6 2020, 3:06 PM
apaskulin triaged this task as Low priority.Oct 6 2020, 3:06 PM

Change 634335 had a related patch set uploaded (by Cicalese; owner: Cicalese):
[mediawiki/skins/WikimediaApiPortal@master] Handle extra user menu and sidebar special pages.

https://gerrit.wikimedia.org/r/634335

Change 634356 had a related patch set uploaded (by Cicalese; owner: Cicalese):
[operations/mediawiki-config@master] Configuration for user menu and sidebar special pages.

https://gerrit.wikimedia.org/r/634356

Change 634335 merged by jenkins-bot:
[mediawiki/skins/WikimediaApiPortal@master] Handle extra user menu and sidebar special pages.

https://gerrit.wikimedia.org/r/634335

Change 634356 merged by jenkins-bot:
[operations/mediawiki-config@master] Configuration for user menu and sidebar special pages.

https://gerrit.wikimedia.org/r/634356

apaskulin added a subscriber: CCicalese_WMF.

@CCicalese_WMF, can you update the description on this task?

Is there a dependence of the extension upon Bootstrap?

apaskulin closed this task as Resolved.Oct 19 2020, 5:06 PM
apaskulin added a subscriber: Art.tsymbar.

Confirmed with @Art.tsymbar that the skin does depend on Bootstrap.

This changed was merged and its effects verified on Beta. Thanks, Cindy!