Jhernandez (Joaquin Oltra Hernandez)
Software engineer. Web developer. Javascript junkie

Projects (7)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Oct 24 2014, 11:57 PM (143 w, 2 d)
Availability
Available
IRC Nick
joakino
LDAP User
Jhernandez
MediaWiki User
Unknown

Recent Activity

Wed, Jul 12

Krinkle awarded T109277: [EPIC]: Use core watchlist code for mobile experience a Orange Medal token.
Wed, Jul 12, 5:33 AM · Reading-Web-Backlog (Tracking), Category, Epic, MobileFrontend

Wed, Jul 5

Jhernandez added a comment to T167713: MinervaNeue's starter modules should we grouped by page not feature.

And if it is smaller than what the team agreed, then all the better!

Wed, Jul 5, 4:51 PM · MinervaNeue, Patch-For-Review, Technical-Debt (RW-Tech-Debt), Reading-Web-Backlog
Jhernandez added a comment to T167713: MinervaNeue's starter modules should we grouped by page not feature.

I'm fine with the task as it is, if we agree on points. We can have big tasks.

Wed, Jul 5, 4:49 PM · MinervaNeue, Patch-For-Review, Technical-Debt (RW-Tech-Debt), Reading-Web-Backlog
Jhernandez updated the image for Marvin from F8624898: profile to F8624901: profile.
Wed, Jul 5, 4:21 PM
Jhernandez updated the image for Marvin from F8624729: fa-briefcase-blue.png to F8624898: profile.
Wed, Jul 5, 4:21 PM
Jhernandez renamed Marvin from Reading-Web-Project-Marvin to Marvin.
Wed, Jul 5, 4:17 PM

Tue, Jul 4

Jhernandez added a comment to T168034: Change autosuggest "wordmatch" style.

1 because:

Tue, Jul 4, 4:49 PM · MW-1.30-release-notes (WMF-deploy-2017-07-18_(1.30.0-wmf.10)), Patch-For-Review, Reading-Web-Kanban-Board, Design (RW-Design-Debt), Reading-Web-Backlog

Mon, Jul 3

Jhernandez updated the task description for T168646: Allow configurable external survey button text.
Mon, Jul 3, 7:38 PM · Patch-Needs-Improvement, Reading-Web-Backlog (Tracking), Patch-For-Review, QuickSurveys
Jhernandez added a comment to T158980: [EPIC] Generate compiled assets from continuous integration.

Sent an email to discuss.

Mon, Jul 3, 3:01 PM · Epic, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

By using runtime it shaves 1/3rd of the size :)

Mon, Jul 3, 2:57 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

@Aleksey_WMDE I didn't know that was the case, I see now in https://vuejs.org/v2/guide/installation.html#Runtime-Compiler-vs-Runtime-only

Mon, Jul 3, 2:49 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog

Fri, Jun 30

Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

The examples from T167041#3394034 but reformatted and + preact and preact(no-jsx):

Fri, Jun 30, 11:58 AM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

The doc definitely addresses the desired outcomes. Is there supposed to be a conclusion here or not?

Fri, Jun 30, 11:04 AM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez moved T168449: event.linkInteractionToken can be undefined from Needs Code Review to Needs More Work on the Reading-Web-Kanban-Board board.
Fri, Jun 30, 10:36 AM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez added a comment to T67079: [Epic] Show what features are included in Beta.

👏

Fri, Jun 30, 10:12 AM · Epic, Reading-Web-Backlog, MobileFrontend
Jhernandez added a comment to T158980: [EPIC] Generate compiled assets from continuous integration.

Seems like a topic for a different card.

Fri, Jun 30, 10:10 AM · Epic, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews

Thu, Jun 29

Jhernandez closed T167181: Create bot that automatically rebases and rebuilds patches to master as Resolved.

Tested it on a patch, seems to work well and fast. Let's see how it serves us, maybe we can revisit in a few weeks to see if we've been using it.

Thu, Jun 29, 11:04 AM · Reading-Web-Kanban-Board, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez closed T167181: Create bot that automatically rebases and rebuilds patches to master, a subtask of T158980: [EPIC] Generate compiled assets from continuous integration, as Resolved.
Thu, Jun 29, 11:04 AM · Epic, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez added a comment to T67079: [Epic] Show what features are included in Beta.

Nice, consistent and clear.

Thu, Jun 29, 10:01 AM · Epic, Reading-Web-Backlog, MobileFrontend

Wed, Jun 28

Jhernandez updated the task description for T168449: event.linkInteractionToken can be undefined.
Wed, Jun 28, 5:19 PM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez added a comment to T167181: Create bot that automatically rebases and rebuilds patches to master.

Will signoff soon.

Wed, Jun 28, 4:08 PM · Reading-Web-Kanban-Board, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez added a comment to T67079: [Epic] Show what features are included in Beta.

@ovasileva @Nirzar For what is worth (I'm a terrible user), I did go through both prototypes, and in the second one I 100% missed the settings cog, and thought we were getting rid of it, and asked myself where would we put the settings like the font size stuff if we got rid of it once things came to stable, and them moved on.

Wed, Jun 28, 12:51 PM · Epic, Reading-Web-Backlog, MobileFrontend
Jhernandez updated the task description for T168625: Make the Page Summary API return an "intro" for a page.
Wed, Jun 28, 12:46 PM · Page-Previews, Reading-Web-Backlog
Jhernandez updated the task description for T168625: Make the Page Summary API return an "intro" for a page.
Wed, Jun 28, 12:46 PM · Page-Previews, Reading-Web-Backlog
Jhernandez added a comment to T168625: Make the Page Summary API return an "intro" for a page.

@ovasileva Any html can be there theoretically. Practically probably not much more.

Wed, Jun 28, 12:45 PM · Page-Previews, Reading-Web-Backlog
Jhernandez added a comment to T168848: Bootstrap an initial version of the Page Summary API in MCS.

Re: Developer notes 2, from an irc chat w/ @phuedx, if this ends up being a REST service, we could consume the parsoid HTML or MCS which have well formed html and sectioning logic already done, instead of using textextracts.

Wed, Jun 28, 10:29 AM · Reading-Web-Kanban-Board, Patch-For-Review, Reading-Infrastructure-Team-Backlog, Mobile-Content-Service, Reading-Web-Backlog, Page-Previews
Jhernandez added a comment to T168625: Make the Page Summary API return an "intro" for a page.

Ping @ovasileva, you have a question in the description.

Wed, Jun 28, 10:18 AM · Page-Previews, Reading-Web-Backlog
Jhernandez updated the task description for T168625: Make the Page Summary API return an "intro" for a page.
Wed, Jun 28, 10:17 AM · Page-Previews, Reading-Web-Backlog
Jhernandez raised the priority of T165036: Add coverage for test scenarios in src/renderer.js unit tests from Lowest to Normal.

Coverage is a vanity metric. Please don't use it to justify priority or imply the code is well tested. It isn't.

Wed, Jun 28, 10:10 AM · Technical-Debt, Need-volunteer, Page-Previews, Reading-Web-Backlog
Jhernandez updated the task description for T168380: Spike: Explore an API for logging events sampled by session.
Wed, Jun 28, 9:53 AM · Reading-Web-Backlog
Jhernandez updated the task description for T168380: Spike: Explore an API for logging events sampled by session.
Wed, Jun 28, 9:45 AM · Reading-Web-Backlog
Jhernandez added a comment to T168380: Spike: Explore an API for logging events sampled by session.

I've clarified title and description, removed the experiment word from the description.

Wed, Jun 28, 9:37 AM · Reading-Web-Backlog
Jhernandez renamed T168380: Spike: Explore an API for logging events sampled by session from Spike: Explore a unifying API for logging by session to Spike: Explore an API for logging events sampled by session.
Wed, Jun 28, 9:33 AM · Reading-Web-Backlog
Jhernandez added a comment to T168380: Spike: Explore an API for logging events sampled by session.

Could we please clarify what is the overall goal here cause I am not sure I understand.

Wed, Jun 28, 9:28 AM · Reading-Web-Backlog
Jhernandez raised the priority of T168625: Make the Page Summary API return an "intro" for a page from Normal to High.

T144622: Add exparagraphs parameter to API has been declined in favor of this task. That one was High prio, so I'm setting this one as High. Feel free to back it down if it isn't.

Wed, Jun 28, 9:14 AM · Page-Previews, Reading-Web-Backlog
Jhernandez closed T144622: Add exparagraphs parameter to API as Declined.

I'm going to go ahead and decline this one given the proposed use case is being tackled in T168625: Make the Page Summary API return an "intro" for a page.

Wed, Jun 28, 9:12 AM · Reading-Web-Backlog, TextExtracts
Jhernandez closed T144622: Add exparagraphs parameter to API, a subtask of T164010: [EPIC] Strengthen the APIs we provide, as Declined.
Wed, Jun 28, 9:12 AM · Epic, Reading-Web-Backlog
Jhernandez added a comment to T167391: [Spike] Isolate the source of Popups event duplication.

@bmansurov For the data about hypothesis 3 (browsers), is there any way we could identify if the duplicate events are sent by a few clients, or many of them? Any anon client token we could group by?

Wed, Jun 28, 9:01 AM · Reading-analysis, Reading-Web-Kanban-Board, Spike, Reading-Web-Backlog, Page-Previews

Tue, Jun 27

Jhernandez moved T167041: Spike: Research JS UI libraries that can render in browser and server from Upcoming to Needs Code Review on the Reading-Web-Kanban-Board board.

I'd appreciate review

Tue, Jun 27, 5:52 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a project to T167041: Spike: Research JS UI libraries that can render in browser and server: Unplanned-Sprint-Work.
Tue, Jun 27, 5:51 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a project to T167041: Spike: Research JS UI libraries that can render in browser and server: Reading-Web-Kanban-Board.
Tue, Jun 27, 5:51 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez updated the task description for T167041: Spike: Research JS UI libraries that can render in browser and server.
Tue, Jun 27, 5:50 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez updated the task description for T167041: Spike: Research JS UI libraries that can render in browser and server.
Tue, Jun 27, 5:49 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a comment to T167022: Inventory requirements for summary 2.0 endpoint across Reading teams.

Sounds great!

Tue, Jun 27, 9:04 AM · Services (watching), Reading-Infrastructure-Team-Backlog (Kanban)
Jhernandez added a comment to T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.

Hold on - please keep in mind that I basically have not been involved with this schema until last month, so this kind of question is perhaps best addressed to the people who were involved with T114303 back in 2015 and/or these later (February 2017) changes ;)

Tue, Jun 27, 9:00 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog

Mon, Jun 26

Jhernandez added a comment to T164033: Test size of "Reading stripped" HTML vs non-stripped HTML.

Very interesting. Seems in line with previous research 👍

Mon, Jun 26, 5:31 PM · Page Content Service, Patch-For-Review, Mobile-Content-Service, Reading-Infrastructure-Team-Backlog (Kanban), Reading Epics (Platform JS CSS and HTML consolidation), Services (designing), RESTBase-API
Jhernandez updated subscribers of T167022: Inventory requirements for summary 2.0 endpoint across Reading teams.

cc/ @phuedx who is working on gathering requirements for the html version of the summary endpoint for Page-Previews.

Mon, Jun 26, 4:48 PM · Services (watching), Reading-Infrastructure-Team-Backlog (Kanban)
Jhernandez added a comment to T164762: Inventory APIs needed for an API driven mobile website.
Mon, Jun 26, 4:44 PM · Reading-Web-Backlog
Jhernandez added a comment to T164762: Inventory APIs needed for an API driven mobile website.

Thanks a lot @Jdlrobson 👏 . Great work

Mon, Jun 26, 4:41 PM · Reading-Web-Backlog
Jhernandez added a comment to T131767: Avoid accidental preferred languages.

IMO sorting the preferred languages by number of previous clicks, would be preferable. Or even showing an (x) to the right of the item to remove it from the list of clicked languages.

Mon, Jun 26, 3:01 PM · Reading-Web-Backlog (Design), MobileFrontend
Jhernandez updated subscribers of T107206: TextExtracts should strip scripts.
Mon, Jun 26, 12:39 PM · Patch-For-Review, Security, Reading-Web-Kanban-Board, Collaboration-Team-Triage, Reading-Web-Backlog, Reading-Web-Sprint-73-One-day-we'll-finish-the-EventLogging-card, TextExtracts
Jhernandez added a comment to T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.

Not being familiar with the feature this is what I see:

I think related articles is enabled for all wikipedia here: https://github.com/wikimedia/operations-mediawiki-config/blob/master/wmf-config/InitialiseSettings.php#L17285
and that whether you are showed more articles is driven by isEnabledForCurrentUser function here: https://github.com/wikimedia/mediawiki-extensions-RelatedArticles/blob/master/resources/ext.relatedArticles.readMore.bootstrap/index.js#L73 which has two buckets on unequal size: 1% and 99%, Thus this code "disables related articles for 1% of population" Terminology is confusing cause it talks about buckets and control when in this case (if I am understanding the code right) is neither, we are just enabling a feature for 99% of devices.

Mon, Jun 26, 12:27 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog
Jhernandez added a comment to T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.

Hi folks, just to double triple check: Can someone verify whether the bucketing and sampling method and rates now match the description on the schema talk page? There seems to be a contradiction regarding sampling rates. (T167310 says 0.01%, but the talk page specification says 1%, based on what I was told last month when I started looking into this whole issue, and vetted later by someone from the team.

Mon, Jun 26, 12:06 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog

Jun 23 2017

Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

I've started work here. I'll bring the summary table back when I'm finished.

Jun 23 2017, 1:44 PM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez claimed T167041: Spike: Research JS UI libraries that can render in browser and server.
Jun 23 2017, 11:09 AM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog

Jun 22 2017

Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

As far as I understood this, preact says it is compatible with React. That "compatibility to react" is what I referred to with "react API" (but maybe, technically, this is not API)

Jun 22 2017, 10:28 AM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog

Jun 21 2017

Jhernandez added a comment to T168371: eventLogging exception in Popups "Value "undefined previews" for property "previewCountBucket" is not one of ...".

Not necessarily undefined I guess, maybe it is being called with NaN or 'NaN' or something like that.

Jun 21 2017, 4:39 PM · Reading-Web-Backlog, Page-Previews
Jhernandez added a subtask for T167835: Log a 'tapped settings cog' event when the footer link is clicked: T168449: event.linkInteractionToken can be undefined.
Jun 21 2017, 4:29 PM · Reading-Web-Backlog, Page-Previews
Jhernandez added a parent task for T168449: event.linkInteractionToken can be undefined: T167835: Log a 'tapped settings cog' event when the footer link is clicked.
Jun 21 2017, 4:29 PM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez updated the task description for T168418: Page Previews should accept the service's notion of emptiness.
Jun 21 2017, 4:22 PM · Reading-Web-Backlog, Page-Previews
Jhernandez added a comment to T167181: Create bot that automatically rebases and rebuilds patches to master.

Yep. I did originally do this for all runs but it's super slow, so I decided the same, just didn't implement it. I want to make sure this is useful before coding it some more.

Jun 21 2017, 12:06 PM · Reading-Web-Kanban-Board, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez added a comment to T168449: event.linkInteractionToken can be undefined.

Thanks @Tbayer, good to know.

Jun 21 2017, 11:29 AM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez added a comment to T167041: Spike: Research JS UI libraries that can render in browser and server.

@Jan_Dittrich There is this optional package preact-compat that shims the React API if you need it.

Jun 21 2017, 11:17 AM · Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-Web-Backlog
Jhernandez added a comment to T168380: Spike: Explore an API for logging events sampled by session.

To be clear, the approach listed on WikimediaEvents is per schema.

Jun 21 2017, 10:55 AM · Reading-Web-Backlog
Jhernandez closed T97988: Maintained cpu usage (60%) on Vagrant managed Virtualbox VM with NFS sharing as Declined.

I'm closing this declined as I haven't experienced this exact behavior for some time and the task is old enough.

Jun 21 2017, 10:34 AM · MediaWiki-Vagrant
Jhernandez closed T97988: Maintained cpu usage (60%) on Vagrant managed Virtualbox VM with NFS sharing, a subtask of T139859: Vagrant and NFS are not always the best of friends [tracking], as Declined.
Jun 21 2017, 10:34 AM · Tracking, MediaWiki-Vagrant
Jhernandez added a comment to T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.

I don't have any particular opinions on the design of the tracking and the test, but just wanted to point out that Hygiene: SamplingRate -> BucketSize is just a name change for the variables, from sampling to bucketing, given that the they are used for bucketing with mw.experiments.

Jun 21 2017, 10:24 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog

Jun 20 2017

Jhernandez added a comment to T167273: pageLoaded events are logged infrequently, if at all.

@pmiazga Nice catch!, but unrelated to this task, and affects probably other events like the settings cog one.

Jun 20 2017, 6:33 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez created T168449: event.linkInteractionToken can be undefined.
Jun 20 2017, 6:31 PM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez updated the task description for T168328: Extracts for Wikimedia List articles display partial previews.
Jun 20 2017, 4:25 PM · Reading-Web-Local-Wiki-Issues, Patch-For-Review, Page-Previews, Reading-Web-Backlog
Jhernandez added a comment to T168328: Extracts for Wikimedia List articles display partial previews.

To be fixed in the service:

Jun 20 2017, 4:22 PM · Reading-Web-Local-Wiki-Issues, Patch-For-Review, Page-Previews, Reading-Web-Backlog
Jhernandez created T168384: Fix codesniffer sniffs disabled with the update to 0.9.
Jun 20 2017, 10:53 AM · MW-1.30-release-notes (WMF-deploy-2017-07-18_(1.30.0-wmf.10)), Patch-For-Review, Reading-Web-Kanban-Board, Need-volunteer, Easy, Reading-Web-Backlog, Page-Previews
phuedx awarded T168380: Spike: Explore an API for logging events sampled by session a Love token.
Jun 20 2017, 10:46 AM · Reading-Web-Backlog
Jhernandez added a comment to T168380: Spike: Explore an API for logging events sampled by session.

I've listed a proposal as an example. It may suck so comment or add other proposals to it.

Jun 20 2017, 10:42 AM · Reading-Web-Backlog
Jhernandez updated the task description for T168380: Spike: Explore an API for logging events sampled by session.
Jun 20 2017, 10:41 AM · Reading-Web-Backlog
Jhernandez added a comment to T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.

I've reviewed but not merged to get @Jdlrobson to have a look at it. LGTM.

Jun 20 2017, 10:41 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog
Jhernandez created T168380: Spike: Explore an API for logging events sampled by session.
Jun 20 2017, 10:37 AM · Reading-Web-Backlog
Jhernandez updated the task description for T167236: userSessionToken in RelatedArticles schema does not seem to survive beyond one pageview.
Jun 20 2017, 10:11 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Reading-analysis, RelatedArticles, Reading-Web-Backlog
Jhernandez added a comment to T168371: eventLogging exception in Popups "Value "undefined previews" for property "previewCountBucket" is not one of ...".

Found when testing T167273: pageLoaded events are logged infrequently, if at all

Jun 20 2017, 9:53 AM · Reading-Web-Backlog, Page-Previews
Jhernandez moved T167273: pageLoaded events are logged infrequently, if at all from Needs Code Review to Ready for Signoff on the Reading-Web-Kanban-Board board.

Needs technical signoff.

Jun 20 2017, 9:52 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez created T168371: eventLogging exception in Popups "Value "undefined previews" for property "previewCountBucket" is not one of ...".
Jun 20 2017, 9:46 AM · Reading-Web-Backlog, Page-Previews
Jhernandez moved T167181: Create bot that automatically rebases and rebuilds patches to master from Needs Code Review to Needs More Work on the Reading-Web-Kanban-Board board.

@Jdlrobson Nice, great job with the docs!

Jun 20 2017, 8:43 AM · Reading-Web-Kanban-Board, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews

Jun 19 2017

Jhernandez closed T168054: Hygiene: Upgrade mw-node-qunit to 2.1.0 as Resolved.
Jun 19 2017, 10:24 AM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Page-Previews

Jun 16 2017

Jhernandez moved T168054: Hygiene: Upgrade mw-node-qunit to 2.1.0 from Upcoming to Needs Code Review on the Reading-Web-Kanban-Board board.
Jun 16 2017, 12:15 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Page-Previews
Jhernandez created T168054: Hygiene: Upgrade mw-node-qunit to 2.1.0.
Jun 16 2017, 12:14 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Patch-For-Review, Unplanned-Sprint-Work, Reading-Web-Kanban-Board, Page-Previews

Jun 15 2017

Jhernandez added a comment to T167787: [Spike 2hr] Investigate ability for page previews in wikidata to appear in user's preferred language.

Added info to description

Jun 15 2017, 6:16 PM · RESTBase-API, Services (blocked), Wikidata, Page-Previews, Reading-Web-Backlog
Jhernandez updated the task description for T167787: [Spike 2hr] Investigate ability for page previews in wikidata to appear in user's preferred language.
Jun 15 2017, 6:15 PM · RESTBase-API, Services (blocked), Wikidata, Page-Previews, Reading-Web-Backlog
Jhernandez added a comment to T165018: Page previews can consume new summary-HTML endpoint.

I've submitted some hygiene changes related to this patch. No change in functionality should occur.

Jun 15 2017, 6:11 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews
Jhernandez updated subscribers of T167787: [Spike 2hr] Investigate ability for page previews in wikidata to appear in user's preferred language.

Discussed in the reading/services sync,

Jun 15 2017, 4:36 PM · RESTBase-API, Services (blocked), Wikidata, Page-Previews, Reading-Web-Backlog
Jhernandez updated subscribers of T166610: Page previews shouldn't be shown when user switches back to a tab.

@Kaartic If that is the issue then it seems easy to fix, nice. jQuery provides event.which, which should be the keyCode normalized across browsers (see .keyup().

Jun 15 2017, 10:36 AM · Reading-Web-Backlog, Page-Previews

Jun 14 2017

Jhernandez updated subscribers of T167272: Incorrect alignment on download as PDF page .

@Nirzar says: Subtitle should be aligned with the title of the checkbox.

Jun 14 2017, 5:33 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Reading-Web-Kanban-Board, Electron-PDFs, Reading-Web-Backlog
Jhernandez added a comment to T167272: Incorrect alignment on download as PDF page .

I see this, is it broken? (Chrome)

Jun 14 2017, 5:32 PM · MW-1.30-release-notes (WMF-deploy-2017-06-20_(1.30.0-wmf.6)), Reading-Web-Kanban-Board, Electron-PDFs, Reading-Web-Backlog
Jhernandez added a comment to T158980: [EPIC] Generate compiled assets from continuous integration.

For the record, I sent an email to wikitech-l asking how does a build process look like for extensions to gather feedback: https://lists.wikimedia.org/pipermail/wikitech-l/2017-June/088264.html

Jun 14 2017, 9:49 AM · Epic, Reading-Web-Backlog, Release-Engineering-Team, Page-Previews
Jhernandez added a comment to T167699: Enable JS error reporting in the mobile website.

@bmansurov Something like Schema:UploadWizardExceptionFlowEvent but instead of the flow* we should probably log the full URL (with hash fragments too).

Jun 14 2017, 9:42 AM · Reading-Web-Kanban-Board, Reading-Web-Backlog, MobileFrontend

Jun 13 2017

Jhernandez added a comment to T167713: MinervaNeue's starter modules should we grouped by page not feature.

Isn't this too big to be just one task?

Jun 13 2017, 10:50 AM · MinervaNeue, Patch-For-Review, Technical-Debt (RW-Tech-Debt), Reading-Web-Backlog
Jhernandez added a comment to T167699: Enable JS error reporting in the mobile website.

@phuedx I've clarified the item to mention both approaches. I believe mobilefrontend heavily uses eventlogging.Schema instances so we should probably follow suit here. What do you think?

Jun 13 2017, 10:46 AM · Reading-Web-Kanban-Board, Reading-Web-Backlog, MobileFrontend
Jhernandez updated the task description for T167699: Enable JS error reporting in the mobile website.
Jun 13 2017, 10:44 AM · Reading-Web-Kanban-Board, Reading-Web-Backlog, MobileFrontend

Jun 12 2017

Jhernandez updated the task description for T167699: Enable JS error reporting in the mobile website.
Jun 12 2017, 5:09 PM · Reading-Web-Kanban-Board, Reading-Web-Backlog, MobileFrontend
Jhernandez created T167699: Enable JS error reporting in the mobile website.
Jun 12 2017, 5:07 PM · Reading-Web-Kanban-Board, Reading-Web-Backlog, MobileFrontend
Jhernandez moved T167626: Regression: Empty extract doesn't trigger generic preview from Doing to Needs Code Review on the Reading-Web-Kanban-Board board.

Tested locally with beta cluster as the api and yorkshire terrier as the sample.

Jun 12 2017, 11:05 AM · MW-1.30-release-notes (WMF-deploy-2017-06-13_(1.30.0-wmf.5)), Patch-For-Review, Unplanned-Sprint-Work, Regression, Reading-Web-Kanban-Board, Reading-Web-Backlog, Page-Previews