| dr0ptp4kt | |
| Aug 26 2016, 7:30 PM |
| F4475559: Screen Shot 2016-09-16 at 10.58.55.png | |
| Sep 16 2016, 2:59 PM |
| F4475545: Screen Shot 2016-09-16 at 10.57.19.png | |
| Sep 16 2016, 2:59 PM |
As a SWAT deployer, I want the ability to update a configuration file and be able to enable Related Articles on one or more wikis on the stable channel mobile web, so that I can roll out Related Articles to the mobile web.
Acceptance criteria:
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T122260 [GOAL] Promote related pages to production (on for everyone) on all wikis | |||
| Resolved | ovasileva | T135030 [GOAL] Roll related pages from beta on mobile web for most wikis | |||
| Resolved | Jdlrobson | T144812 Deploy related pages to mobile web stable channel - part 1 | |||
| Resolved | • Jhernandez | T144047 Add support for enabling Related Articles on mobile web stable channel |
@dr0ptp4kt - do we need an extra task for the actual move to stable or is it included here?
An extra task or set of tasks is needed for actual move to stable. In T143164: Create rollout plan for related pages on mobile and propose to community it appears a rollout strategy is being considered. This tech is going to be hitting our APIs pretty hard, so you probably want to do it in waves just in case. You should sync up with the engineers to figure out what is both easy in the configuration files yet allows for a wave rollout.
Change 310381 had a related patch set uploaded (by Bmansurov):
Allow blacklisting skins for showing ReadMore in footer
Change 311051 had a related patch set uploaded (by Bmansurov):
Hygiene: don't load a module unless necessary
@ovasileva, @Nirzar, I noticed that after allowing related articles to seen on Mobile web stable, the related articles could use some design love too. This is how it currently looks:
Notice how the articles are not stretched to the page width or how thumbnails have whitespace at the bottom.
Compare it to how related articles look on mobile web beta:
Should we create a follow up task or re-purpose this one to make some design changes?
When implementing this [the footer] with Nirzar the plan was to release the new footer to stable before pushing related articles.
They're in reverse order now. Would it be possible to just introduce spacing for now until the footer moves to stable. @Nirzar - any other suggestions?
WAIT.. why is it broken? the broken things have nothing to with new footer?
what changed? didn't we have this before without new footer?
When implementing this [the footer] with Nirzar the plan was to release the new footer to stable before pushing related articles.
this is true too but as i said, it should not matter for the issues i mentioned above.
okay after my outburst for alignments here ^ @Jdlrobson calmed me down ;)
here's what we talked
bottom line, we cannot release related pages with the issues that we are seeing here and huge thanks for pointed them out @bmansurov
over to @ovasileva to choose the course of action
strong recommendation for releasing new footer and related pages together.
I agree that dependency on the footer is a blocker for related pages. If we decide T144579: Beta footer appears obscured on diff pages is not a blocker for the footer, we can potentially bring the footer work into this sprint. Let's discuss during standup.
Change 311195 had a related patch set uploaded (by Bmansurov):
Beta: Allow displaying Related Articles in the footer
@bmansurov: There are test failures and a minor explanatory comment from yours truly on rEMFRa606ffab03ce: Beta: Allow displaying Related Articles in the footer.
Left comments on https://gerrit.wikimedia.org/r/#/c/311195/7 and https://gerrit.wikimedia.org/r/#/c/310381/3 - should be easily remedied so please ping me when you need me to do a follow up review!
Change 311195 merged by jenkins-bot:
Beta: Allow displaying Related Articles in the footer
Change 310381 merged by jenkins-bot:
Allow blacklisting skins for showing ReadMore in footer