Fjalapeno (Corey Floyd)
Engineering Manager, WMF Readers

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Jan 16 2015, 4:46 PM (162 w, 2 d)
Availability
Available
LDAP User
Fjalapeno
MediaWiki User
Unknown

Recent Activity

Wed, Feb 21

Fjalapeno added a comment to T187892: Individual errors for batch operations.

@JoeWalsh is the big issue for you about having to try 500 individual requests if the entire batch fails fail?

Wed, Feb 21, 8:50 PM · Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno edited projects for T187892: Individual errors for batch operations, added: Reading List Service, Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.
Wed, Feb 21, 3:17 PM · Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno added a comment to T187892: Individual errors for batch operations.

@Tgr given your plan to implement batching internally (insert all records at once), is this still possible?

Wed, Feb 21, 3:16 PM · Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno created T187884: Replace GeoHack with Kartographer.
Wed, Feb 21, 2:25 PM · Maps-Sprint

Wed, Feb 14

Fjalapeno added a comment to T187277: Should it be possible for a schema to override DNT in exceptional circumstances?.

Yeah it sounds like we are trying to use EL “the wrong way”

Wed, Feb 14, 9:34 PM · Analytics-EventLogging, Analytics

Mon, Feb 12

Fjalapeno added a comment to T186933: 204 summary responses cause Android app to crash.

I think not using any 204 No Content responses would be the quickest solution. We could use 200s and provide a special value in the type field instead, e.g. something like none. If necessary we could revisit the 204s for a 2.0 release.

Agreed. AIUI the current Page Previews implementation will handle HTTP 200s with empty or non-existent extract properties the same way. The HTTP 204 response was meant to formalise this at the server level.

Mon, Feb 12, 4:26 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T186933: 204 summary responses cause Android app to crash.

@Fjalapeno Sounds reasonable. I moved Android's prio meeting to Wednesday to accommodate the meeting you set up today, but if @Dbrant will make a ticket then let's get it done asap. I accept that it's high priority because it's blocking switchover, but it's also just good practice to handle all HTTP codes gracefully, so it's worth doing anyway.

Mon, Feb 12, 4:21 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno updated subscribers of T186933: 204 summary responses cause Android app to crash.

Ok, I'll chat about this with Sam today, but I think we have some pretty clear things to do:

Mon, Feb 12, 3:06 PM · Patch-For-Review, Services (doing), RESTBase-API, RESTBase, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Wed, Feb 7

Fjalapeno assigned T186296: Increase item limit to 5k - and return limit information in API to Tgr.
Wed, Feb 7, 4:21 PM · Patch-For-Review, Reading List Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno added a comment to T182706: Make sure apps can continue /changes/since where they left off .

@Tgr it seems like syncing for apps can be simplified to just using changes since endpoint, if we enable this behavior of passing "0" (distant past). The other REST APIs are still needed for Web and other applications, but this should at least solve the issue of getting the first time stamp.

Wed, Feb 7, 4:05 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service

Tue, Feb 6

Fjalapeno added a project to T184000: Magic word on English WP to override display of Wikidata short description: Reading-Infrastructure-Team-Backlog (Kanban).
Tue, Feb 6, 6:13 PM · Reading-Infrastructure-Team-Backlog (Kanban), MediaWiki-extensions-WikibaseClient, Wikidata, Community-Tech

Fri, Feb 2

Fjalapeno placed T186296: Increase item limit to 5k - and return limit information in API up for grabs.
Fri, Feb 2, 4:48 PM · Patch-For-Review, Reading List Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno assigned T186296: Increase item limit to 5k - and return limit information in API to Tgr.
Fri, Feb 2, 3:25 AM · Patch-For-Review, Reading List Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno created T186296: Increase item limit to 5k - and return limit information in API.
Fri, Feb 2, 3:25 AM · Patch-For-Review, Reading List Service, Reading-Infrastructure-Team-Backlog (Kanban)

Thu, Feb 1

Fjalapeno closed T182840: kartotherian package repo fails to build as Resolved.
Thu, Feb 1, 8:09 PM · Services (next), service-runner, Maps-Sprint, Maps (Maps-data)
Fjalapeno closed T182840: kartotherian package repo fails to build, a subtask of T162241: Deploy meddo / cleartables as part of tilerator / kartotherian, as Resolved.
Thu, Feb 1, 8:08 PM · Services (watching), Patch-For-Review, Maps-Sprint, Maps (Maps-data)

Wed, Jan 31

Fjalapeno updated subscribers of T182706: Make sure apps can continue /changes/since where they left off .

@JoeWalsh @Dbrant if we can't do a token and we must have timestamps, does @Tgr's minimum solution seem ok? Is knowing that you are getting duplicates hurt app logic in any way?

Wed, Jan 31, 4:26 AM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno added a comment to T182706: Make sure apps can continue /changes/since where they left off .

Thanks for putting all this together…

Wed, Jan 31, 4:23 AM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service

Tue, Jan 30

Fjalapeno edited projects for T186041: Return full objects from POST and PUT endpoints for reading lists and reading list entries, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.

Cool, thanks moving to the Kanban board then

Tue, Jan 30, 8:12 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno assigned T186041: Return full objects from POST and PUT endpoints for reading lists and reading list entries to Tgr.

@Tgr this came out of the need for supporting client side ordering - this seemed reasonable to me, does it look ok to you as well?

Tue, Jan 30, 7:42 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno added a project to T186041: Return full objects from POST and PUT endpoints for reading lists and reading list entries: Reading List Service.
Tue, Jan 30, 7:40 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service
Fjalapeno awarded T185607: Provide an inline discussion feature, "DiscussThis" a Love token.
Tue, Jan 30, 3:48 AM · VisualEditor-MediaWiki-Plugins, Collaboration-Team-Triage, StructuredDiscussions, VisualEditor, TechCom-RFC
Fjalapeno added a comment to T183313: Wikimedia Developer Summit 2018 Topic: Evolving the MediaWiki Architecture.

Here are some distilled notes from the session.

Tue, Jan 30, 3:24 AM · MediaWiki-Platform-Team, User-Jdlrobson, Services (watching), User-mobrovac, User-Daniel, Wikimedia-Developer-Summit (2018)

Mon, Jan 29

Beliefunleashed awarded T148574: Update black list a Like token.
Mon, Jan 29, 5:20 AM · Reading-Infrastructure-Team-Backlog, Unplanned-Sprint-Work, Mobile-App-Android-Sprint-94-Plutonium, Patch-For-Review, Easy, Mobile-Content-Service

Jan 17 2018

Fjalapeno added a watcher for Code-Stewardship-Reviews: Fjalapeno.
Jan 17 2018, 8:30 PM
Fjalapeno added a comment to T183313: Wikimedia Developer Summit 2018 Topic: Evolving the MediaWiki Architecture.

Some opt my own thoughts and some cribbing from the ATWG sessions:

Jan 17 2018, 3:52 PM · MediaWiki-Platform-Team, User-Jdlrobson, Services (watching), User-mobrovac, User-Daniel, Wikimedia-Developer-Summit (2018)
Fjalapeno updated the task description for T183313: Wikimedia Developer Summit 2018 Topic: Evolving the MediaWiki Architecture.
Jan 17 2018, 3:32 PM · MediaWiki-Platform-Team, User-Jdlrobson, Services (watching), User-mobrovac, User-Daniel, Wikimedia-Developer-Summit (2018)

Jan 16 2018

Fjalapeno moved T173821: Investigate exposing content styles needed via API vs as HTML tags from Doing to Blocked on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Jan 16 2018, 3:27 PM · RESTBase-API, Services (watching), Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service
Fjalapeno added a comment to T173821: Investigate exposing content styles needed via API vs as HTML tags.

Waiting on CSS development to start… blocking on that

Jan 16 2018, 3:27 PM · RESTBase-API, Services (watching), Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service
Fjalapeno moved T175735: [Spike] Investigate developing a Share-a-fact card service in MCS from Doing to To Do on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Jan 16 2018, 3:26 PM · RESTBase-API, Services (watching), Wikipedia-iOS-App-Backlog, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno edited projects for T172148: Determine URL paths for Zim files, added: Reading-Infrastructure-Team-Backlog; removed Reading-Infrastructure-Team-Backlog (Kanban).
Jan 16 2018, 3:26 PM · Reading-Infrastructure-Team-Backlog, Operations, Traffic, Wikipedia-Android-App-Backlog, Android-app-feature-Compilations
Fjalapeno edited projects for T172137: Estimate both beta and production downloads of Zim files, added: Reading-Infrastructure-Team-Backlog; removed Reading-Infrastructure-Team-Backlog (Kanban).
Jan 16 2018, 3:25 PM · Reading-Infrastructure-Team-Backlog, Reading-analysis, Wikipedia-Android-App-Backlog, Android-app-feature-Compilations

Jan 10 2018

Fjalapeno added a comment to T91162: RFC: Shadow namespaces.

@Fjalapeno Shadow namespaces don't really do anything for easier parsing,

Jan 10 2018, 10:31 PM · Wikimedia-Developer-Summit (2017), TechCom-RFC
Fjalapeno added a comment to T91162: RFC: Shadow namespaces.

@daniel potentially… there is definitely a desire to be able to reliably parse information from main pages, news portals, features article pages, etc…

Jan 10 2018, 4:58 PM · Wikimedia-Developer-Summit (2017), TechCom-RFC
Fjalapeno closed T181718: compilation of mapnik node module fails with new kartotherian packaging repo, a subtask of T162241: Deploy meddo / cleartables as part of tilerator / kartotherian, as Resolved.
Jan 10 2018, 2:09 AM · Services (watching), Patch-For-Review, Maps-Sprint, Maps (Maps-data)
Fjalapeno closed T181718: compilation of mapnik node module fails with new kartotherian packaging repo as Resolved.
Jan 10 2018, 2:09 AM · Patch-For-Review, Maps-Sprint
Fjalapeno closed T160215: Clean up mess with Kartotherian and Tilerator configs as Resolved.
Jan 10 2018, 2:09 AM · Patch-For-Review, Technical-Debt, Maps-Sprint, Maps (Kartotherian)
Fjalapeno closed T181808: Kartotherian error with new deployment: "core.strToFloat is not a function" as Resolved.
Jan 10 2018, 2:09 AM · Maps (Kartotherian), Maps-Sprint
Fjalapeno closed T181808: Kartotherian error with new deployment: "core.strToFloat is not a function", a subtask of T162241: Deploy meddo / cleartables as part of tilerator / kartotherian, as Resolved.
Jan 10 2018, 2:08 AM · Services (watching), Patch-For-Review, Maps-Sprint, Maps (Maps-data)

Jan 9 2018

Fjalapeno closed T160639: Package ClearTables to deploy it on maps servers as Resolved.
Jan 9 2018, 8:05 PM · Maps-Sprint, Maps (Maps-data)
Fjalapeno closed T160639: Package ClearTables to deploy it on maps servers, a subtask of T156682: Deploying new vector tiles to production, as Resolved.
Jan 9 2018, 8:05 PM · Maps-Sprint, Maps (Maps-data)
Fjalapeno closed T182583: maps-test2001 is low on disk space as Resolved.
Jan 9 2018, 8:05 PM · Discovery, Maps, Operations, Maps-Sprint
Fjalapeno added a comment to T183588: Change the user-created list name if it the same name as the default list.

Ok, that didn't take long… I am re-wrong again 🤦‍♂️

Jan 9 2018, 7:35 PM · Patch-For-Review, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Reading Epics (Synchronized Reading Lists)
Fjalapeno added a comment to T183588: Change the user-created list name if it the same name as the default list.

@RHo I talked to the iOS team about this today also. I had a misunderstanding of how the Default lists were identified by the server. They are identified simply by the name property, so the service does need to make sure that it rejects setting the name to "Default" for any list that is not the actual default list. Thanks for being on top of this…

Jan 9 2018, 7:25 PM · Patch-For-Review, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Reading Epics (Synchronized Reading Lists)
Fjalapeno added a comment to T177512: Ensure all APIs include the Time UUID and the Revision ID in the response payload.

Yes, this is now supposed to be in the payload. Good catch - feel free to update and repurpose this ticket for that

Jan 9 2018, 5:54 PM · Reading-Infrastructure-Team-Backlog (Kanban), Page Content Service, Reading Epics (Platform JS CSS and HTML consolidation)

Jan 8 2018

Fjalapeno added a comment to T184327: Add section header to references output.

@bearND something we didn't discus and should be covered here, but just to highlight:

Jan 8 2018, 8:06 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)

Jan 5 2018

Fjalapeno edited projects for T182052: Batch reading list operations, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.
Jan 5 2018, 4:46 PM · MW-1.31-release-notes (WMF-deploy-2018-01-16 (1.31.0-wmf.17)), Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Reading List Service

Jan 3 2018

Fjalapeno added a comment to T183588: Change the user-created list name if it the same name as the default list.

@RHo the server really doesn't care about list names ad @Tgr mentioned.

Jan 3 2018, 10:49 PM · Patch-For-Review, Wikipedia-Android-App-Backlog (Android-app-release-v2.7.23x-G), Reading Epics (Synchronized Reading Lists)

Dec 20 2017

Fjalapeno added a project to T183049: Reference anchor and target IDs use inconsistent percent encoding: Reading-Infrastructure-Team-Backlog.
Dec 20 2017, 6:31 PM · Readers-Web-Backlog (Tracking), Reading-Infrastructure-Team-Backlog, MobileFrontend

Dec 13 2017

Fjalapeno created T182802: Update announcements to stop English fundraiser on December 20, 2017.
Dec 13 2017, 5:23 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban)

Dec 11 2017

Fjalapeno added a comment to T122942: RFC: Support language variants in the REST API.

@C933103 Just an FYI, this is only for APIs, not for user facing web URLs.

Dec 11 2017, 3:25 PM · Reading-Infrastructure-Team-Backlog, User-mobrovac, Chinese-Sites, Services (next), I18n, RfC, Mobile-Content-Service, Parsing-Team, TechCom-RFC, RESTBase, Blocked-on-Parsoid, Parsoid

Dec 7 2017

Fjalapeno closed T179116: Develop rough architecture diagram, a subtask of T179046: [epic] Investigate Mapzen open source vector tiles, as Resolved.
Dec 7 2017, 8:21 PM · Epic, Maps-Sprint, Discovery
Fjalapeno closed T179116: Develop rough architecture diagram as Resolved.
Dec 7 2017, 8:21 PM · Maps-Sprint, Discovery

Dec 5 2017

Fjalapeno moved T179116: Develop rough architecture diagram from Needs review to Done on the Maps-Sprint board.
Dec 5 2017, 8:20 PM · Maps-Sprint, Discovery

Dec 4 2017

Fjalapeno added a comment to T180092: Finalize reading list metadata schema.

@Tgr dropping them is all cool… those were old design ideas that never materialized…

Dec 4 2017, 10:33 PM · MW-1.31-release-notes (WMF-deploy-2017-11-28 (1.31.0-wmf.10)), Patch-For-Review, Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Reading-Infrastructure-Team-Backlog, Reading List Service

Nov 30 2017

Fjalapeno updated subscribers of T181516: Perform Security Review of Commons Android App Authentication Logic.

@NHarateh_WMF @Dbrant I saw this got put into "tracking", but it is work that the Android team needs to perform. Just want to make sure that this gets scheduled and doesn't get lost in the shuffle.

Nov 30 2017, 4:05 PM · Commons, WMF-Legal, Wikipedia-Android-App-Backlog

Nov 29 2017

Fjalapeno added a comment to T181004: Run Big English fundraising on apps.

Just a note deployment, if we want to get this deployed this week, we need to finalize everything today or tonight. We have 2 deployments left:

Nov 29 2017, 5:15 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Fjalapeno added a comment to T181004: Run Big English fundraising on apps.

@schoenbaechler We can update the button text, there is no facility to update the button style in the apps.

Nov 29 2017, 5:00 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Fjalapeno added a member for Maps-Sprint: Fjalapeno.
Nov 29 2017, 4:11 PM

Nov 28 2017

Fjalapeno added a comment to T181004: Run Big English fundraising on apps.

@bearND yes

Nov 28 2017, 7:17 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Fjalapeno updated subscribers of T181004: Run Big English fundraising on apps.

@schoenbaechler we are working on implementing this, but @JMinor is out today and want to make sure he reviews the text as well.

Nov 28 2017, 7:17 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Fjalapeno edited projects for T181004: Run Big English fundraising on apps, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.
Nov 28 2017, 6:50 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Wikimedia-Fundraising, Mobile-Content-Service
Fjalapeno added a comment to T69015: m.wikipedia.org and zero.wikipedia.org should redirect how/where.

@Mholloway spoke with @JoeWalsh about the timing and targeting first week of January seems reasonable. Can you check back in with @JoeWalsh that week to make sure we are a go?

Nov 28 2017, 6:46 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Zero, Discovery, Mobile, Wikimedia-Apache-configuration
Fjalapeno added a project to T69015: m.wikipedia.org and zero.wikipedia.org should redirect how/where: Wikipedia-iOS-App-Backlog.
Nov 28 2017, 4:55 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Zero, Discovery, Mobile, Wikimedia-Apache-configuration
Fjalapeno created T181517: Perform Security Review of Commons Android App Authentication Logic.
Nov 28 2017, 3:59 PM · Commons, WMF-Legal, Wikipedia-Android-App-Backlog
Fjalapeno created T181516: Perform Security Review of Commons Android App Authentication Logic.
Nov 28 2017, 3:59 PM · Commons, WMF-Legal, Wikipedia-Android-App-Backlog
Fjalapeno added a comment to T180635: Delete reading list items by name.

@Tgr thanks for hopping on this. We already talked, but writing here to document: Batching is blocking for client adoption, but you don't need to block pushing the service to production, if it is easier to think about the batching solution first. Lets be sure to get it in ASAP either way, the app teams will need this API to do testing - which will be performed this month and next with a projected production release in January.

Nov 28 2017, 3:49 PM · Reading List Service, Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Reading-Infrastructure-Team-Backlog

Nov 27 2017

Fjalapeno updated the task description for T180384: Turn off Trending Service.
Nov 27 2017, 7:57 PM · Services (done), User-Joe, Operations, Reading-Infrastructure-Team-Backlog (Kanban), Trending-Service
Fjalapeno added a comment to T180384: Turn off Trending Service.

Hey all, following up here after the holiday.

Nov 27 2017, 7:31 PM · Services (done), User-Joe, Operations, Reading-Infrastructure-Team-Backlog (Kanban), Trending-Service

Nov 15 2017

Fjalapeno added a comment to T152284: Today’s Featured Article and In the News is not shown in Hungarian mobile app.

No, the news is unavailable we probably need top split up this ticket. Since featured article is resolved but news is not.

Nov 15 2017, 4:09 AM · Mobile-Content-Service, Reading-Infrastructure-Team-Backlog, Reading Epics (Feed Content Language Expansion), Wikipedia-Android-App-Backlog, Android-app-feature-Feeds

Nov 14 2017

Fjalapeno added projects to T166092: Copyrighted images in Explore feed: Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog.

Tagging app projects for visibility

Nov 14 2017, 4:33 PM · Wikipedia-Android-App-Backlog, Wikipedia-iOS-App-Backlog, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service

Nov 13 2017

Fjalapeno added a comment to T180402: Remove custom ordering from ReadingLists extension.

Some answers…

Nov 13 2017, 8:31 PM · MW-1.31-release-notes (WMF-deploy-2017-11-28 (1.31.0-wmf.10)), Patch-For-Review, Reading-Infrastructure-Team-Backlog, Reading List Service
Fjalapeno added a comment to T180384: Turn off Trending Service.

@mobrovac that is not any type of assurance that would happen… sorry if that is confusing. We should not plan on this returning to production.

Nov 13 2017, 7:03 PM · Services (done), User-Joe, Operations, Reading-Infrastructure-Team-Backlog (Kanban), Trending-Service
Fjalapeno created T180384: Turn off Trending Service.
Nov 13 2017, 6:03 PM · Services (done), User-Joe, Operations, Reading-Infrastructure-Team-Backlog (Kanban), Trending-Service

Nov 9 2017

Fjalapeno added a comment to T177431: Develop a Summary JSON API.

@Mholloway something I just noticed… sorry for all the last minute things…

Nov 9 2017, 8:46 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

@Fjalapeno, @bearND: Re T177431#3748630 and T177431#3748749: It might be worth defaulting to the "base" editing experience (no hash fragment in the URL) for now while we think about how we should approach linking to the mobile editing experience. Right now, my suggestion of the base URL plus the hash fragment seems more of a fortunate hack rather than an actual solution.

Nov 9 2017, 8:45 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

I imagine we should also change these names in the formatted and formatted-lead (next-gen mobile-sections) endpoints, too, then?

Nov 9 2017, 8:42 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno closed T170694: Define JSON API or APIs to return everything structured that a client needs to display a page that isn't in the summary or references API as Resolved.
Nov 9 2017, 8:30 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno moved T170694: Define JSON API or APIs to return everything structured that a client needs to display a page that isn't in the summary or references API from Doing to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Nov 9 2017, 8:30 PM · Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

I'm leaning towards removing the reference to mobile-sections since we will consider this a deprecated API once PCS is functional. No need to proliferate usage at this point IMO.

Nov 9 2017, 6:30 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

By the way, did we decide for sure on using .../wiki/Title?action=edit#/editor/0 as the mobile edit URL, even though it refers specifically to the lead section?

Nov 9 2017, 6:29 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.
  1. We need the mobile and desktop URLs in the content_URLs

https://gerrit.wikimedia.org/r/#/c/389584/ (blocked on an addition to siteinfo via MobileFrontend: https://gerrit.wikimedia.org/r/#/c/390248/)

Nov 9 2017, 6:28 PM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno closed T170581: Determine if the PCS needs to perform HTML transformations in a separate layer as Resolved.
Nov 9 2017, 4:12 PM · Services (designing), wikimedia-page-library, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno moved T170581: Determine if the PCS needs to perform HTML transformations in a separate layer from Doing to Deployed on the Reading-Infrastructure-Team-Backlog (Kanban) board.
Nov 9 2017, 4:12 PM · Services (designing), wikimedia-page-library, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban)
Fjalapeno edited projects for T179786: Update trending-edits' node-rdkafka to v1.x, added: Reading-Infrastructure-Team-Backlog (Kanban); removed Reading-Infrastructure-Team-Backlog.
Nov 9 2017, 4:12 PM · Patch-For-Review, User-Joe, Operations, Wikimedia-Incident, Reading-Infrastructure-Team-Backlog (Kanban), User-Jdlrobson, Trending-Service, Services (watching)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

@Mholloway @bearND One more bikeshed…

Nov 9 2017, 5:52 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

@phuedx Hey looking at the spec for type… we have the following:

Nov 9 2017, 5:49 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

@Mholloway can you confirm the comment I just made? Does that seem right as a list of things to do?

Nov 9 2017, 5:40 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)
Fjalapeno added a comment to T177431: Develop a Summary JSON API.

So the things I see that we need to do (if the response is up to date)

Nov 9 2017, 5:40 AM · Patch-For-Review, Page Content Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation)

Nov 7 2017

Fjalapeno added a comment to T179519: Implementing OAuth in Wikimedia Commons Android app.

@Tgr just to follow up on your comment on the need to use OAuth: The requirement comes from legal... basically as way to remove the liability of being responsible for handling user credentials.

Nov 7 2017, 2:11 AM · Commons-App-Android, Commons, MediaWiki-extensions-OAuth
Fjalapeno updated subscribers of T179519: Implementing OAuth in Wikimedia Commons Android app.

I’m adding @Anomie here who has a lot of insight into authorization in MediaWiki
Brad, do you have any insight into the issue here? Seems like that code is pointing to an invalid OAuth Key

Nov 7 2017, 2:08 AM · Commons-App-Android, Commons, MediaWiki-extensions-OAuth

Nov 6 2017

Fjalapeno updated the task description for T170692: Return common URLs in summary API so clients do not have to perform bug prone string manipulation .
Nov 6 2017, 8:52 PM · MW-1.31-release-notes (WMF-deploy-2017-11-14 (1.31.0-wmf.8)), Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Page Content Service, Services (watching)
Fjalapeno added a comment to T170692: Return common URLs in summary API so clients do not have to perform bug prone string manipulation .

From my perspective this would be handled better by a dedicated node library (similar to mediawiki-title) since you can construct those urls from information you already have. On the other hand doing it in the server will add a bloat to all responses…

Nov 6 2017, 8:52 PM · MW-1.31-release-notes (WMF-deploy-2017-11-14 (1.31.0-wmf.8)), Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Page Content Service, Services (watching)

Nov 3 2017

Fjalapeno added a comment to T170692: Return common URLs in summary API so clients do not have to perform bug prone string manipulation .
Surely, the client can easily work out these particular URLs and may choose to ignore them altogether by providing it's own experience?
Nov 3 2017, 9:13 PM · MW-1.31-release-notes (WMF-deploy-2017-11-14 (1.31.0-wmf.8)), Reading-Infrastructure-Team-Backlog (Kanban), Patch-For-Review, Page Content Service, Services (watching)
Fjalapeno created T179735: Audit and document points of user consent in Desktop Web (Not VE).
Nov 3 2017, 9:02 PM · Readers-Web-Backlog (Tracking), WMF-Legal, Reading Epics
Fjalapeno created T179734: Audit and document points of user consent in Visual Editor.
Nov 3 2017, 9:00 PM · VisualEditor, WMF-Legal, Reading Epics
Fjalapeno added projects to T179733: Audit and document points of user consent in MobileFrontend: MobileFrontend, Readers-Web-Backlog.
Nov 3 2017, 8:59 PM · Readers-Web-Backlog (Tracking), MobileFrontend, WMF-Legal, Reading Epics
Fjalapeno added a project to T179731: Audit and document points of user consent in the iOS App: Wikipedia-iOS-App-Backlog.
Nov 3 2017, 8:58 PM · WMF-Legal, Reading Epics
Fjalapeno added a project to T179732: Audit and document points of user consent in Android App: Wikipedia-Android-App-Backlog.
Nov 3 2017, 8:57 PM · Wikipedia-Android-App-Backlog, WMF-Legal, Reading Epics
Fjalapeno created T179733: Audit and document points of user consent in MobileFrontend.
Nov 3 2017, 8:57 PM · Readers-Web-Backlog (Tracking), MobileFrontend, WMF-Legal, Reading Epics