Page MenuHomePhabricator

Update page issues folder structure
Open, NormalPublic2 Story Points

Description

As of https://gerrit.wikimedia.org/r/#/c/mediawiki/skins/MinervaNeue/+/475576/, page issues code is now split between:

  • resources/skins.minerva.scripts (pageIssues.js and pageIssuesParser.js)
  • tests/qunit/skins.minerva.scripts (pageIssues.test.js and pageIssuesParser.test.js)
  • resources/skins.minerva.scripts/page-issues

Refactor the remaining code to move it under the page-issues subdirectory.

Acceptance criteria

  • All page issues production and test code lives under resources/skins.minerva.scripts/page-issues/ and tests/qunit/skins.minerva.scripts/page-issues/
  • Rename PageIssuesOverlay.less to pageIssuesOverlay.less to match pageIssuesOverlay.js.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 4 2019, 3:25 PM
Jdlrobson assigned this task to nray.
Jdlrobson added subscribers: nray, Jdlrobson.

@nray can you take a look at this task and confirm it's clear what would be expected of you if you were to work on it and thus ready for estimation?

Jdlrobson triaged this task as Normal priority.Jan 11 2019, 12:53 AM
nray added a comment.Jan 12 2019, 12:05 AM

I plan to carve out time to look at this on Monday, Jan. 14

nray updated the task description. (Show Details)Jan 14 2019, 11:03 PM
nray updated the task description. (Show Details)Jan 14 2019, 11:17 PM
nray updated the task description. (Show Details)
nray updated the task description. (Show Details)Jan 14 2019, 11:37 PM
nray removed nray as the assignee of this task.Jan 14 2019, 11:58 PM
Jdlrobson moved this task from Backlog to Bugs on the MinervaNeue board.Feb 5 2019, 6:49 PM
Jdlrobson moved this task from Bugs to Tech debt on the MinervaNeue board.
ovasileva set the point value for this task to 2.Apr 10 2019, 4:39 PM

@ovasileva, @Jdlrobson, just a friendly heads up that we discussed this today and think it's ready to work on