Page MenuHomePhabricator
Create Task
Maniphest T208921

MFA: Port mobile.watchstar and mobile.pagelist.scripts code to webpack
Closed, ResolvedPublic5 Estimated Story Points
Actions

Description

Code relating to the watchstar code currently lives in mobile.watchstar and mobile.pagelist.scripts. Beaming with the success of T207805, we will move it to webpack.

Acceptance criteria

  • There is no mobile.watchstar modules-these are now part of mobile.startup ! yay! (Thus: the watchstar code is loaded in the initial JS bundle for Minerva) and there are no deprecation warnings in the console
  • There is no mobile.pagelist.scripts modules -these are now part of mobile.startup ! yay! (Thus: the watchstar code is loaded in the initial JS bundle for Minerva)
  • Tests for mobile.pagelist.scripts and mobile.watchstar can be run in headless mode
  • References to M.require( 'mobile.pagelist.scripts/WatchstarPageList' ) have been removed in Mobilefrontend and there are no deprecation warnings in the console
  • Tests in tests/qunit/mobile.watchstar/ and tests/qunit/mobile.pagelist.scripts/ have been removed
  • resources/mobile.watchstar/watchstar.less has been relocated to resources/mobile.startup/watchstar.less to reflect which module it is tied to
  • Minerva is retained in a mergeable state throughout the migration.

Sign off steps

  • Progress test coverage is updated.
  • Make sure "Automate asset bundling" table is up to date in Progress
  • Make sure acceptance criteria all signed off.

Details

ProjectBranchLines +/-Subject
mediawiki/extensions/MobileFrontendmaster+2 -11Hygiene: move pageList.less to reflect src/mobile.startup
mediawiki/skins/MinervaNeuemaster+2 -2mobile.watchstar is a deprecated module
mediawiki/extensions/MobileFrontendmaster+3 -220Watchstar webpack migration follow ups
mediawiki/extensions/MobileFrontendmaster+0 -195Drop redundant tests
mediawiki/extensions/MobileFrontendmaster+1 K -601Moving Watchstar, WatchstarGateway & WatchstarPagelist to webpack
Customize query in gerrit

Related Objects
Search...

Event Timeline

Jdlrobson created this task.Nov 7 2018, 12:52 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 7 2018, 12:52 AM
Jdlrobson moved this task from Needs Prioritization to Upcoming on the Readers-Web-Backlog board.Nov 7 2018, 12:52 AM
Jdlrobson added a parent task: T207787: [EPIC] Reduce the amount of modules in MobileFrontend.
Jdlrobson mentioned this in T207805: MFA: Port the references feature to webpack: Merge mobile.references and mobile.references.gateway.Nov 7 2018, 12:55 AM
Jdlrobson moved this task from Upcoming to Triaged but Future on the Readers-Web-Backlog board.Nov 9 2018, 9:27 PM
Jdlrobson renamed this task from Port mobile.watchstar and mobile.pagelist.scripts code to webpack to MFA: Port mobile.watchstar and mobile.pagelist.scripts code to webpack.Nov 15 2018, 6:59 PM
Jdlrobson triaged this task as High priority.
Jdlrobson moved this task from Triaged but Future to Upcoming on the Readers-Web-Backlog board.Nov 16 2018, 6:26 PM
Restricted Application added a subscriber: • Gilles. · View Herald TranscriptNov 20 2018, 3:48 PM
Jdlrobson set the point value for this task to 5.Nov 20 2018, 5:52 PM
Jdlrobson added a project: MobileFrontend (MobileFrontend and MinervaNeue architecture).Nov 20 2018, 7:51 PM
Jdlrobson moved this task from Upcoming to Readers-Web-Kanbanana-Board-2018-19-Q2 on the Readers-Web-Backlog board.Nov 21 2018, 12:05 AM
Jdlrobson edited projects, added Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2); removed Readers-Web-Backlog.
Jdrewniak claimed this task.Nov 21 2018, 2:38 PM
Jdrewniak moved this task from To Do to Blocked on Others on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Jdrewniak moved this task from Blocked on Others to Doing on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
gerritbot added a subscriber: gerritbot.Nov 26 2018, 3:20 PM

Change 475766 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/extensions/MobileFrontend@master] Moving Watchstar, WatchstarGateway & WatchstarPagelist to webpack

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

gerritbot added a project: Patch-For-Review.Nov 26 2018, 3:20 PM
Jdlrobson moved this task from Doing to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 26 2018, 5:26 PM
Jdlrobson moved this task from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 26 2018, 9:41 PM
Niedzielski added a parent task: T210207: MFA: Port search related code to webpack.Nov 27 2018, 5:15 PM
Jdlrobson moved this task from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 27 2018, 7:02 PM
Jdlrobson moved this task from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 28 2018, 12:02 AM
Jdlrobson moved this task from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Nov 28 2018, 6:24 PM
Jdlrobson claimed this task.Nov 28 2018, 6:51 PM
Jdlrobson added a subscriber: Jdrewniak.

Suspect that this will get merged today once T204876 is working locally for me...

gerritbot added a comment.Nov 28 2018, 11:13 PM

Change 475766 merged by Jdlrobson:
[mediawiki/extensions/MobileFrontend@master] Moving Watchstar, WatchstarGateway & WatchstarPagelist to webpack

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

Jdlrobson moved this task from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.EditedNov 28 2018, 11:43 PM
Jdlrobson removed a project: Patch-For-Review.

There are 3 patches needed to follow up this:

  1. references to mobile.watchstar in Minerva that will need to be updated to mobile.startup
  2. mobile.watchstar should be removed from MobileFrontend.
  3. tests/qunit/mobile.pagelist.scripts/WatchstarPageList.test.js references a deprecated module
Jdlrobson updated the task description. (Show Details)Nov 28 2018, 11:44 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson removed Jdlrobson as the assignee of this task.Nov 28 2018, 11:48 PM
Jdlrobson updated the task description. (Show Details)
ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.8; 2018-12-11).Nov 29 2018, 12:01 AM
• Gilles removed a subscriber: • Gilles.Nov 29 2018, 12:13 PM
Jdlrobson updated the task description. (Show Details)Nov 30 2018, 1:11 AM
Jdlrobson updated the task description. (Show Details)
gerritbot added a comment.Nov 30 2018, 1:20 AM

Change 476793 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Drop redundant tests

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

gerritbot added a project: Patch-For-Review.Nov 30 2018, 1:20 AM
Jdlrobson updated the task description. (Show Details)Nov 30 2018, 1:21 AM
gerritbot added a comment.Nov 30 2018, 3:22 PM

Change 476793 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Drop redundant tests

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

Jdlrobson removed a project: Patch-For-Review.Nov 30 2018, 3:49 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson updated the task description. (Show Details)Nov 30 2018, 11:24 PM
gerritbot added a comment.Nov 30 2018, 11:28 PM

Change 476995 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] mobile.watchstar is a deprecated module

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

gerritbot added a project: Patch-For-Review.Nov 30 2018, 11:28 PM
gerritbot added a comment.Nov 30 2018, 11:35 PM

Change 476998 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Watchstar webpack migration follow ups

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

Jdlrobson added a comment.Dec 1 2018, 12:27 AM

^ We should look to fix the deprecation warnings before the end of Monday as that's going to lead to a bunch of deprecation warnings in production😱.
Leaving in your capable hands!

gerritbot added a comment.Dec 3 2018, 4:26 PM

Change 476998 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Watchstar webpack migration follow ups

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

Niedzielski added a subscriber: Niedzielski.Dec 3 2018, 9:14 PM

Should the following be removed from extension.json?

		"mobile.watchstar": {
			"targets": [
				"mobile",
				"desktop"
			],
			"deprecated": true,
			"dependencies": [
				"mobile.startup"
			]
		}

I don't see one for the references module which had a similar port.

gerritbot added a comment.Dec 3 2018, 9:35 PM

Change 476995 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] mobile.watchstar is a deprecated module

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

Niedzielski added a comment.Dec 3 2018, 9:45 PM

I think this should be moved too:

  • ./resources/mobile.startup/watchstar.less -> resources/mobile.startup/watchstar/

I'm not sure if this task covers watchlist:

  • resources/mobile.watchlist -> src/mobile.startup/watchlist
  • ./resources/mobile.special.watchlist.scripts -> src/mobile.startup/watchlist
  • ./tests/qunit/mobile.watchlist/ -> tests/node-qunit/mobile.startup/watchlist
Niedzielski added a comment.Dec 3 2018, 9:49 PM

Er, and I think this pagelist less?

  • ./resources/mobile.pagelist.styles => mobile.startup.styles/pagelist/
Jdrewniak added a comment.Dec 10 2018, 10:15 AM

https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/477827/
took care of watchstar.less, thanks @Niedzielski

I believe mobile.watchlist/ is not part of this patch.

gerritbot added a comment.Dec 10 2018, 10:15 AM

Change 478627 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/extensions/MobileFrontend@master] Hygiene: move pageList.less to reflect src/mobile.startup

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

Jdrewniak updated the task description. (Show Details)Dec 10 2018, 10:17 AM
Jdrewniak moved this task from Needs More Work to Needs Code Review on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.
Niedzielski moved this task from Needs Code Review to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.Dec 11 2018, 4:42 PM
Jdlrobson added a comment.Dec 11 2018, 6:35 PM

Er, and I think this pagelist less?
./resources/mobile.pagelist.styles => mobile.startup.styles/pagelist/

Nope this should stay where it is. It is used in addModuleStyles calls so should remain and be JS only:
ag T208921-followup x ~/git/core/extensions/MobileFrontend $ ag mobile.pagelist.styles
extension.json
85: "mobile.pagelist.styles": {
91: "resources/mobile.pagelist.styles/pagelist.less"
282: "mobile.pagelist.styles",

includes/specials/SpecialMobileContributions.php
64: 'mobile.pagelist.styles',

includes/specials/SpecialMobileEditWatchlist.php
223: 'mobile.pagelist.styles',

includes/specials/SpecialMobileHistory.php
112: 'mobile.pagelist.styles',

includes/specials/SpecialMobileWatchlist.php
50: 'mobile.pagelist.styles',

Jdlrobson claimed this task.Dec 11 2018, 6:36 PM
Jdlrobson moved this task from Needs More Work to Ready for Signoff on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.

This looks done to me and I can sign off today.

Jdlrobson updated the task description. (Show Details)Dec 12 2018, 12:02 AM
Jdlrobson closed this task as Resolved.Dec 12 2018, 12:05 AM

Signing off. Folders are clear and wiki page updated.

gerritbot added a comment.Dec 14 2018, 12:59 AM

Change 478627 abandoned by Jdlrobson:
Hygiene: move pageList.less to reflect src/mobile.startup

Reason:
Lemme know if I'm missing something!

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

Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL