Initial version of article rendering
Closed, DeclinedPublic
Actions

Assigned To

None

Authored By

	• Jhernandez
	Aug 14 2017, 3:14 PM

Description

Details

Subject	Repo	Branch	Lines +/-
Update: use content header margins for section headers	marvin	master	+16 -12
New: add random page and summary endpoints	marvin	master	+126 -56
Update: add revision and time ID support to page summary client	marvin	master	+39 -17
Chore: break apart page modules	marvin	master	+64 -69
Update: add namespace model	marvin	master	+72 -0
Chore: move page redirect to distinct type	marvin	master	+15 -14

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Declined	ovasileva	T176958 Display article content within article namespace
Declined	None	T173317 Initial version of article rendering
Resolved	• Niedzielski	T173324 Add page route and rendering
Resolved	• Niedzielski	T173326 Extract page title component from /summary and use it on /wiki
Duplicate	None	T173327 Render sections
Resolved	• Niedzielski	T173328 Add content page section component and render all sections
Resolved	• Niedzielski	T173330 Add content styles
Resolved	• Niedzielski	T173321 Add MCS data client
Resolved	• Jhernandez	T173740 POC for universal page component with data fetching
Declined	None	T177681 Page URL should update on redirect
Declined	• bearND	T179600 Add canonical title to mobile-sections lead
Resolved	• Niedzielski	T178616 Move wiki revision path parameter to query parameter
Resolved	• Niedzielski	T178615 Add query parameter typing
Resolved	• Niedzielski	T178617 Add query parameter parser

Event Timeline

• Jhernandez created this task.Aug 14 2017, 3:14 PM

• Jhernandez moved this task from Needs Triage to Current Epics (2017-Q2) on the Marvin board.Aug 14 2017, 3:19 PM

• Jhernandez updated the task description. (Show Details)

• Niedzielski created subtask T173321: Add MCS data client.Aug 14 2017, 3:22 PM

• Niedzielski created subtask T173324: Add page route and rendering.Aug 14 2017, 4:01 PM

• Niedzielski removed a subtask: T173321: Add MCS data client.Aug 14 2017, 7:02 PM

• Niedzielski updated the task description. (Show Details)

• Jhernandez created subtask T173740: POC for universal page component with data fetching.Aug 21 2017, 4:11 PM

• Jhernandez closed subtask T173740: POC for universal page component with data fetching as Resolved.Sep 6 2017, 5:59 PM

• Jhernandez mentioned this in T173323: Add Preact libraries.Sep 18 2017, 8:55 AM

Change 381373 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] Update: add namespace model

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

gerritbot added a project: Patch-For-Review.Sep 29 2017, 2:37 AM

• Niedzielski mentioned this in rMARV8c3931e85602: Update: add namespace model.Sep 29 2017, 2:38 AM

Change 381374 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] Chore: move page redirect to distinct type

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

• Niedzielski mentioned this in rMARV329db077b81d: Chore: move page redirect to distinct type.Sep 29 2017, 2:44 AM

• Niedzielski added a parent task: T176958: Display article content within article namespace.Sep 29 2017, 12:52 PM

• Niedzielski mentioned this in rMARV8a9841800299: Update: add namespace model.Sep 29 2017, 5:25 PM

• Niedzielski mentioned this in rMARV85170d9e63dc: Chore: move page redirect to distinct type.

• Niedzielski mentioned this in rMARVeacb47cacf14: Update: add namespace model.Sep 29 2017, 5:32 PM

• Niedzielski mentioned this in rMARVe9f4de05ce2e: Chore: move page redirect to distinct type.

• Niedzielski mentioned this in rMARVcd2d72ef52be: Update: add namespace model.Oct 2 2017, 1:27 PM

• Niedzielski mentioned this in rMARVb6af1db96e07: Chore: move page redirect to distinct type.

• Niedzielski mentioned this in rMARVef2659d2f7b4: Chore: move page redirect to distinct type.Oct 2 2017, 1:30 PM

• Niedzielski mentioned this in rMARV36ae0d0e4912: Update: add namespace model.

• Niedzielski mentioned this in rMARVa2009ac65136: Update: add namespace model.Oct 2 2017, 1:38 PM

Change 381374 merged by jenkins-bot:
[marvin@master] Chore: move page redirect to distinct type

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

Change 381373 merged by jenkins-bot:
[marvin@master] Update: add namespace model

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

Change 381913 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] Chore: break apart page modules

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

Change 381915 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] Update: add revision and time ID support to page summary client

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

• Niedzielski mentioned this in rMARV04f78f025565: Chore: break apart page modules.Oct 3 2017, 1:03 AM

• Niedzielski mentioned this in rMARV3aec5ae61608: Update: add revision and time ID support to page summary client.

Change 381913 merged by jenkins-bot:
[marvin@master] Chore: break apart page modules

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

Change 381915 merged by jenkins-bot:
[marvin@master] Update: add revision and time ID support to page summary client

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

Change 381993 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] New: add dummy wiki page

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

• Niedzielski mentioned this in rMARV23555ba64ed9: New: add dummy wiki page.Oct 3 2017, 2:44 PM

@Niedzielski would you mind adding the patches to the subtasks of the Epic? For example rMARV23555ba64ed9: New: add dummy wiki page could be a fit for T173324: Add page route and rendering.

I don't mind having patches in an epic if it isn't split into subtasks, I think that is totally fine, but this one is split so maybe we should use those subtasks instead.

@Jhernandez, fixed. I'll try to be better about this going forward!

Change 383659 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] New: add random page and summary endpoints

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

• Niedzielski mentioned this in rMARVfe0e870cafb9: New: add random page and summary endpoints.Oct 11 2017, 8:02 PM

• Niedzielski closed subtask T173324: Add page route and rendering as Resolved.Oct 12 2017, 3:44 PM

• Niedzielski reopened subtask T173324: Add page route and rendering as Open.

• Niedzielski mentioned this in rMARV979d10864f54: New: add random page and summary endpoints.Oct 17 2017, 1:49 PM

• Jhernandez added a subtask: T177681: Page URL should update on redirect.Oct 18 2017, 11:13 AM

• Niedzielski mentioned this in rMARVaef0c7499355: New: add random page and summary endpoints.Oct 18 2017, 1:27 PM

Change 383659 merged by jenkins-bot:
[marvin@master] New: add random page and summary endpoints

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

• Niedzielski closed subtask T173324: Add page route and rendering as Resolved.Oct 18 2017, 10:10 PM

• Niedzielski created subtask T178615: Add query parameter typing.Oct 19 2017, 7:26 PM

• Niedzielski created subtask T178616: Move wiki revision path parameter to query parameter.Oct 19 2017, 7:31 PM

• Niedzielski changed the status of subtask T177681: Page URL should update on redirect from Open to Stalled.Nov 2 2017, 5:59 PM

• Niedzielski changed the status of subtask T177681: Page URL should update on redirect from Stalled to Open.Nov 7 2017, 7:40 PM

• Niedzielski changed the status of subtask T177681: Page URL should update on redirect from Open to Stalled.Nov 9 2017, 1:37 PM

• Niedzielski removed a project: Patch-For-Review.Nov 17 2017, 5:24 PM

Change 397902 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[marvin@master] Update: use content header margins for section headers

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

gerritbot added a project: Patch-For-Review.Dec 12 2017, 7:42 PM

• Niedzielski mentioned this in rMARV1a6c1f53b832: Update: use content header margins for section headers.Dec 12 2017, 7:45 PM

• Niedzielski mentioned this in rMARV335300994982: Update: use content header margins for section headers.Dec 15 2017, 8:25 PM

@Jhernandez, let's close this epic once all the open patches are merged. T177681 is important but stalled on PCS changes. This epic isn't adding a lot of value in itself and we can keep track of T177681 on the board by itself.