[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Remove ExtMobileFrontend::eventLog

2018-01-17 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/404703 ) Change subject: Hygiene: Remove ExtMobileFrontend::eventLog .. Hygiene: Remove ExtMobileFrontend::eventLog Since I25f9e1ca, the method is

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Drop use of MobileOptionsTracking schema

2018-01-16 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/404495 ) Change subject: Drop use of MobileOptionsTracking schema .. Drop use of MobileOptionsTracking schema Going forward we can rely on pageview

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Revision: Handle all return values of Title::newFromId

2017-12-27 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/400408 ) Change subject: Revision: Handle all return values of Title::newFromId .. Revision: Handle all return values of Title::newFromId In

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: instrumentation: Bump schema revision

2017-12-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398293 ) Change subject: instrumentation: Bump schema revision .. instrumentation: Bump schema revision Following on from I6643168a, the client-side

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Update MobileContext#logMobileMode documentation

2017-12-13 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/398116 ) Change subject: Update MobileContext#logMobileMode documentation .. Update MobileContext#logMobileMode documentation ... with a note to

[MediaWiki-commits] [Gerrit] integration/config[master]: Add npm job for the Chromium render service

2017-11-29 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394058 ) Change subject: Add npm job for the Chromium render service .. Add npm job for the Chromium render service The Chromium render service is

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: gateway/rest: Handle large "small" thumbnails

2017-10-12 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/383831 ) Change subject: gateway/rest: Handle large "small" thumbnails .. gateway/rest: Handle large "small" thumbnails NOTE: This is a test case to

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Register PHPUnit tests via UnitTestsList hook

2017-10-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/383350 ) Change subject: Register PHPUnit tests via UnitTestsList hook .. Register PHPUnit tests via UnitTestsList hook Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Collection[master]: Don't request render if renderer not configured

2017-10-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/383347 ) Change subject: Don't request render if renderer not configured .. Don't request render if renderer not configured If

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: subscriber: Wait until browser idle before logging

2017-09-21 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/379532 ) Change subject: subscriber: Wait until browser idle before logging .. subscriber: Wait until browser idle before logging If no additional

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Stop A/B test on enwiki and dewiki

2017-09-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/378686 ) Change subject: pagePreviews: Stop A/B test on enwiki and dewiki .. pagePreviews: Stop A/B test on enwiki and dewiki Changes: * Remove A/B

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Scale A/B test bucket sizes by 10

2017-08-30 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/374815 ) Change subject: pagePreviews: Scale A/B test bucket sizes by 10 .. pagePreviews: Scale A/B test bucket sizes by 10 Currently, we're seeing

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Bump on/control group size to 25% (BC-only)

2017-08-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373541 ) Change subject: pagePreviews: Bump on/control group size to 25% (BC-only) .. pagePreviews: Bump on/control group size to 25% (BC-only) Bug:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Re-enable on Beta Cluster

2017-08-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373531 ) Change subject: pagePreviews: Re-enable on Beta Cluster .. pagePreviews: Re-enable on Beta Cluster I85c3fcfd accidentally disabled Page

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Enable A/B test (BC-only)

2017-08-23 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373264 ) Change subject: pagePreviews: Enable A/B test (BC-only) .. pagePreviews: Enable A/B test (BC-only) Enable anonymous user bucket on all

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: relatedArticles: Tidy up config

2017-08-22 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/373043 ) Change subject: relatedArticles: Tidy up config .. relatedArticles: Tidy up config The sidebar feature has been removed, meaning that

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Deploy to next 100 stage 1 wikis

2017-08-16 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372137 ) Change subject: pagePreviews: Deploy to next 100 stage 1 wikis .. pagePreviews: Deploy to next 100 stage 1 wikis Note well that the dewiki

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Deploy to first 50 of stage 1 wikis

2017-08-03 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/369960 ) Change subject: pagePreviews: Deploy to first 50 of stage 1 wikis .. pagePreviews: Deploy to first 50 of stage 1 wikis Add the pp_stage2

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Increase i13n sampling rate for ruwiki

2017-07-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/367398 ) Change subject: pagePreviews: Increase i13n sampling rate for ruwiki .. pagePreviews: Increase i13n sampling rate for ruwiki Correct the

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Increase instrumentation sampling rate

2017-07-21 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/366882 ) Change subject: pagePreviews: Increase instrumentation sampling rate .. pagePreviews: Increase instrumentation sampling rate ... for

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Revert "Stop RelatedArticles A/B test and clean up c...

2017-07-20 Thread Phuedx (Code Review)
Hello jenkins-bot, Dereckson, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/366546 to review the following change. Change subject: Revert "Revert "Stop RelatedArticles A/B test and clean up config""

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: i13n: popupEnabled = false for disabled event

2017-07-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/366287 ) Change subject: i13n: popupEnabled = false for disabled event .. i13n: popupEnabled = false for disabled event ... for consistency with the

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Enable page previews for everyone on labs"

2017-07-18 Thread Phuedx (Code Review)
Hello Florianschmidtwelzow, jenkins-bot, Jdlrobson, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/365943 to review the following change. Change subject: Revert "Enable page previews for everyone on labs"

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Revert "Log all events for page previews in beta cluster"

2017-07-18 Thread Phuedx (Code Review)
Hello Florianschmidtwelzow, jenkins-bot, Jdlrobson, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/365941 to review the following change. Change subject: Revert "Log all events for page previews in beta cluster"

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Re-enable Popups extension on Beta Cluster

2017-07-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/365939 ) Change subject: pagePreviews: Re-enable Popups extension on Beta Cluster .. pagePreviews: Re-enable Popups extension on Beta Cluster

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[wmf/1.30.0-wmf.7]: Don't subscribe EventLogging twice if window.onload fires twice

2017-07-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/364389 ) Change subject: Don't subscribe EventLogging twice if window.onload fires twice .. Don't subscribe EventLogging twice if window.onload fires

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Remove wgMFTidyMobileViewSections

2017-07-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/364230 ) Change subject: Hygiene: Remove wgMFTidyMobileViewSections .. Hygiene: Remove wgMFTidyMobileViewSections wgMFTidyMobileViewSections was

[MediaWiki-commits] [Gerrit] mediawiki...EventLogging[master]: WIP

2017-07-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/364188 ) Change subject: WIP .. WIP Change-Id: I348079ee80e1d27f9dd70c0340237edc8d0371b0 --- M modules/ext.eventLogging.subscriber.js 1 file

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: i13n: Return false over not sampling

2017-07-03 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/362977 ) Change subject: Hygiene: i13n: Return false over not sampling .. Hygiene: i13n: Return false over not sampling Previously, if the browser

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: i13n: Update MobileOptionsTracking schema rev

2017-06-27 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/361643 ) Change subject: i13n: Update MobileOptionsTracking schema rev .. i13n: Update MobileOptionsTracking schema rev The images property has now

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: Hygiene: SamplingRate -> BucketSize

2017-06-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/360165 ) Change subject: Hygiene: SamplingRate -> BucketSize .. Hygiene: SamplingRate -> BucketSize In order to avoid further confusion, clarify

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: relatedArticles: SamplingRate -> BucketSize

2017-06-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/360166 ) Change subject: relatedArticles: SamplingRate -> BucketSize .. relatedArticles: SamplingRate -> BucketSize In order to avoid further

[MediaWiki-commits] [Gerrit] mediawiki...RelatedArticles[master]: i13n: Don't sample by pageview

2017-06-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/359921 ) Change subject: i13n: Don't sample by pageview .. i13n: Don't sample by pageview Currently, the mw.eventLog.Schema class samples per

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: experiments: Extract mediawiki.hash class

2017-06-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/359918 ) Change subject: experiments: Extract mediawiki.hash class .. experiments: Extract mediawiki.hash class mw.experiments uses a private

[MediaWiki-commits] [Gerrit] mediawiki/vagrant[master]: RelatedArticles no longer depends on Cards

2017-06-15 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/359193 ) Change subject: RelatedArticles no longer depends on Cards .. RelatedArticles no longer depends on Cards Since I784fd132, the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: Group instrumentation modules

2017-06-15 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/359130 ) Change subject: Hygiene: Group instrumentation modules .. Hygiene: Group instrumentation modules Following on from I4f653bba, since the

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Consume HTML from RESTBase endpoint

2017-06-15 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/359123 ) Change subject: pagePreviews: Consume HTML from RESTBase endpoint .. pagePreviews: Consume HTML from RESTBase endpoint ... on the Beta

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: i13n: Log EL events with mw.track

2017-06-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/358957 ) Change subject: i13n: Log EL events with mw.track .. i13n: Log EL events with mw.track Currently, the mw.eventLog.Schema class samples per

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: i13n: Extract experiments module

2017-06-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/358956 ) Change subject: i13n: Extract experiments module .. i13n: Extract experiments module ... from the statsvInstrumentation module so that the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[wmf/1.30.0-wmf.4]: actions/rest: Use DB-key version of title

2017-06-13 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/358528 ) Change subject: actions/rest: Use DB-key version of title .. actions/rest: Use DB-key version of title This reduces the number of 301

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: actions/rest: Use DB-key version of title

2017-06-12 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/358338 ) Change subject: actions/rest: Use DB-key version of title .. actions/rest: Use DB-key version of title This reduces the number of 301

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: eventLogging: Add missing properties to "tapped settings cog...

2017-06-07 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/357570 ) Change subject: eventLogging: Add missing properties to "tapped settings cog" event .. eventLogging: Add missing properties to "tapped

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: gateway: Fix Accept header sent by rest gateway

2017-06-06 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/357359 ) Change subject: gateway: Fix Accept header sent by rest gateway .. gateway: Fix Accept header sent by rest gateway Prior to this change the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document reducers/eventLogging module

2017-05-26 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355749 ) Change subject: doc: Document reducers/eventLogging module .. doc: Document reducers/eventLogging module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: eventLogging: Add perceivedWait prop to all events

2017-05-25 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355646 ) Change subject: eventLogging: Add perceivedWait prop to all events .. eventLogging: Add perceivedWait prop to all events ... and the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document userSettings module

2017-05-25 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355634 ) Change subject: doc: Document userSettings module .. doc: Document userSettings module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Add doc:start script

2017-05-25 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355607 ) Change subject: doc: Add doc:start script .. doc: Add doc:start script Supporting changes: * Use nodemon@1.11.0. Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP doc: Document gateway modules

2017-05-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355474 ) Change subject: WIP doc: Document gateway modules .. WIP doc: Document gateway modules Additional changes: * Fix the summary not showing up

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document statsvInstrumentation module

2017-05-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355401 ) Change subject: doc: Document statsvInstrumentation module .. doc: Document statsvInstrumentation module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: mobileFrontend: Move first paragraph before infobox

2017-05-24 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355397 ) Change subject: mobileFrontend: Move first paragraph before infobox .. mobileFrontend: Move first paragraph before infobox Following on

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document counts module

2017-05-23 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355199 ) Change subject: doc: Document counts module .. doc: Document counts module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document isEnabled module

2017-05-23 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355200 ) Change subject: doc: Document isEnabled module .. doc: Document isEnabled module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: docs: Document schema module

2017-05-22 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355145 ) Change subject: docs: Document schema module .. docs: Document schema module Bug: T158236 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document wait module

2017-05-22 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355144 ) Change subject: doc: Document wait module .. doc: Document wait module Bug: T158236 Change-Id: I5bdd3d7bb6fd0eb219b01ae113dc5bc8bdf12d0a

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: doc: Document preview/model module

2017-05-22 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/355143 ) Change subject: doc: Document preview/model module .. doc: Document preview/model module Changes: * Assign exports to exports rather than

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP Make JSDuck Work Again

2017-05-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/354455 ) Change subject: WIP Make JSDuck Work Again .. WIP Make JSDuck Work Again Change-Id: I8da712075dd28f6b228c96fa9577ca39eb148fb5 --- M

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: rl: Break out mobile.mainMenu style module

2017-05-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/354225 ) Change subject: rl: Break out mobile.mainMenu style module .. rl: Break out mobile.mainMenu style module The mobile.mainMenu styles are

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Hygiene: Break out mobile.messageBox styles module

2017-05-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/354222 ) Change subject: Hygiene: Break out mobile.messageBox styles module .. Hygiene: Break out mobile.messageBox styles module Because the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: actions: Mix title and namespaceID into LINK_DWELL

2017-05-15 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/353903 ) Change subject: actions: Mix title and namespaceID into LINK_DWELL .. actions: Mix title and namespaceID into LINK_DWELL This fixes a bug

[MediaWiki-commits] [Gerrit] mediawiki...Popups[wmf/1.29.0-wmf.21]: eventLogging: Discard events with duplicate tokens

2017-05-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/353058 ) Change subject: eventLogging: Discard events with duplicate tokens .. eventLogging: Discard events with duplicate tokens I6a38a261 made the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[wmf/1.30.0-wmf.1]: eventLogging: Discard events with duplicate tokens

2017-05-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/353038 ) Change subject: eventLogging: Discard events with duplicate tokens .. eventLogging: Discard events with duplicate tokens I6a38a261 made the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: eventLogging: Discard events with duplicate tokens

2017-05-10 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/353035 ) Change subject: eventLogging: Discard events with duplicate tokens .. eventLogging: Discard events with duplicate tokens I6a38a261 made the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP eventLogging: Add missing *Hover properties

2017-05-09 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/352870 ) Change subject: WIP eventLogging: Add missing *Hover properties .. WIP eventLogging: Add missing *Hover properties Bug: T164256 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: eventLogging: Round perceivedWait property

2017-05-09 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/352868 ) Change subject: eventLogging: Round perceivedWait property .. eventLogging: Round perceivedWait property The perceivedWait property,

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: Tidy up QUnit references

2017-05-04 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/351798 ) Change subject: Hygiene: Tidy up QUnit references .. Hygiene: Tidy up QUnit references Since Ieea378c9 all QUnit tests are run in Node.js

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: eventLogging: Add missing perceivedWait property

2017-05-03 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/351605 ) Change subject: eventLogging: Add missing perceivedWait property .. eventLogging: Add missing perceivedWait property Per the Popups schema

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Create pp_stage0.dblist

2017-05-02 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/351286 ) Change subject: pagePreviews: Create pp_stage0.dblist .. pagePreviews: Create pp_stage0.dblist Extract the repeated set of stage 0 wikis

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Deploy to first 50 of stage 1 wikis

2017-05-02 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/351287 ) Change subject: pagePreviews: Deploy to first 50 of stage 1 wikis .. pagePreviews: Deploy to first 50 of stage 1 wikis Add the pp_stage2

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: Document removeXMLProcInst line in .svgo.yml

2017-04-28 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/350814 ) Change subject: Document removeXMLProcInst line in .svgo.yml .. Document removeXMLProcInst line in .svgo.yml Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Remove isLoggingEnabled with Null Object pattern

2017-04-27 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/350552 ) Change subject: Remove isLoggingEnabled with Null Object pattern .. Remove isLoggingEnabled with Null Object pattern The statsv change

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Optimise SVGs with svgo

2017-04-26 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/350395 ) Change subject: Optimise SVGs with svgo .. Optimise SVGs with svgo ... by running the following: svgo --disable removeXMLProcInst

[MediaWiki-commits] [Gerrit] operations/puppet[production]: Reading Web Page Previews alerts

2017-04-26 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/350377 ) Change subject: Reading Web Page Previews alerts .. Reading Web Page Previews alerts Use the Nagios Grafana alerts adapter introduced in

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Track and discard duplicate enqueued events

2017-04-25 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/350206 ) Change subject: Track and discard duplicate enqueued events .. Track and discard duplicate enqueued events The eventLogging change listener

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Don't show preview when user abandons link

2017-04-21 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/349410 ) Change subject: Don't show preview when user abandons link .. Don't show preview when user abandons link Bug: T163350 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Don't occlude link when preview is above mouse

2017-04-20 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/349231 ) Change subject: Don't occlude link when preview is above mouse .. Don't occlude link when preview is above mouse When the user dwells on a

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Foo

2017-04-19 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348919 ) Change subject: Foo .. Foo Change-Id: I1acdfc15237d1d91e144b9fa5933ab6b3cfd9186 --- M tests/node-qunit/actions.test.js 1 file changed, 5

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Don't destroy interaction on LINK_CLICK

2017-04-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348789 ) Change subject: reducers: Don't destroy interaction on LINK_CLICK .. reducers: Don't destroy interaction on LINK_CLICK I09d8776 introduced

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: actions: Conditionally dispatch ABANDON_*

2017-04-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348790 ) Change subject: actions: Conditionally dispatch ABANDON_* .. actions: Conditionally dispatch ABANDON_* If the user CmdOrCtrl+Clicks on a

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP

2017-04-18 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348780 ) Change subject: WIP .. WIP Bug: T162924 Change-Id: I340dee6881437b254c53b187c15a1adad7769b27 --- M resources/dist/index.js M

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Make LINK_CLICK finalize but not close

2017-04-16 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348321 ) Change subject: reducers: Make LINK_CLICK finalize but not close .. reducers: Make LINK_CLICK finalize but not close ... the interaction.

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Update eventLogging documentation

2017-04-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348266 ) Change subject: reducers: Update eventLogging documentation .. reducers: Update eventLogging documentation Bug: T159490 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: dist: Bundle assets with webpack@2.4.1

2017-04-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348226 ) Change subject: dist: Bundle assets with webpack@2.4.1 .. dist: Bundle assets with webpack@2.4.1 Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP

2017-04-14 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348200 ) Change subject: WIP .. WIP Change-Id: If1ee1853ba7a9b2a66b24bb93b4e6062b92b0dba --- M includes/PopupsHooks.php 1 file changed, 4

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: WIP Revert "Upgrade to jQuery v3"

2017-04-13 Thread Phuedx (Code Review)
Hello Krinkle, Jack Phoenix, jenkins-bot, Jforrester, I'd like you to do a code review. Please visit https://gerrit.wikimedia.org/r/348068 to review the following change. Change subject: WIP Revert "Upgrade to jQuery v3"

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: WIP

2017-04-13 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/348058 ) Change subject: WIP .. WIP Change-Id: Id5315b67ddd0e6be6cae5176377acda13f8dd9bb --- M api.php 1 file changed, 1 insertion(+), 1 deletion(-)

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[wmf/1.29.0-wmf.20]: formatter: Change log channel of infobox message

2017-04-12 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347886 ) Change subject: formatter: Change log channel of infobox message .. formatter: Change log channel of infobox message By default, the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Make LINK_CLICK finalize interaction

2017-04-12 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347829 ) Change subject: reducers: Make LINK_CLICK finalize interaction .. reducers: Make LINK_CLICK finalize interaction ... in the eventLogging

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: formatter: Change log level of infobox message

2017-04-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347802 ) Change subject: formatter: Change log level of infobox message .. formatter: Change log level of infobox message I61d20ea was only

[MediaWiki-commits] [Gerrit] mediawiki...MobileFrontend[master]: formatter: Change log channel of infobox message

2017-04-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347619 ) Change subject: formatter: Change log channel of infobox message .. formatter: Change log channel of infobox message By default, the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: actions: Add token to PREVIEW_SHOW

2017-04-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347598 ) Change subject: actions: Add token to PREVIEW_SHOW .. actions: Add token to PREVIEW_SHOW Mirroring all other actions that are dispatched

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Make PREVIEW_SHOW require a token

2017-04-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347600 ) Change subject: reducers: Make PREVIEW_SHOW require a token .. reducers: Make PREVIEW_SHOW require a token Like the FETCH_COMPLETE and

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: DRY up eventLogging reducer

2017-04-11 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347599 ) Change subject: Hygiene: DRY up eventLogging reducer .. Hygiene: DRY up eventLogging reducer Extract the repeated token testing for the

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: pagePreviews: Enable NavPopups gadget detection

2017-04-09 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347291 ) Change subject: pagePreviews: Enable NavPopups gadget detection .. pagePreviews: Enable NavPopups gadget detection Note well that the name

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Remove dependency on es5-shim RL module

2017-04-08 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/347148 ) Change subject: Remove dependency on es5-shim RL module .. Remove dependency on es5-shim RL module Per I95400637, the es5-shim module is

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: Reduce nesting of test cases

2017-04-07 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346981 ) Change subject: Hygiene: Reduce nesting of test cases .. Hygiene: Reduce nesting of test cases The ABANDON_START action test cases were

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: reducers: Reduce FETCH_COMPLETE if token matches

2017-04-07 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346979 ) Change subject: reducers: Reduce FETCH_COMPLETE if token matches .. reducers: Reduce FETCH_COMPLETE if token matches ... instead of using

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: WIP

2017-04-07 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346970 ) Change subject: WIP .. WIP Change-Id: Ic71dd3ce5e7933273f84a9a64d41e7f3a4cb03f4 --- M resources/dist/index.js M resources/dist/index.js.map

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: Hygiene: Remove trailing whitespace

2017-04-06 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346834 ) Change subject: Hygiene: Remove trailing whitespace .. Hygiene: Remove trailing whitespace ... from tests/node-qunit/actions.test.js.

[MediaWiki-commits] [Gerrit] mediawiki...Popups[wmf/1.29.0-wmf.19]: actions: Correctly delay FETCH_COMPLETE

2017-04-06 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346832 ) Change subject: actions: Correctly delay FETCH_COMPLETE .. actions: Correctly delay FETCH_COMPLETE I496fe317 caused a regression where the

[MediaWiki-commits] [Gerrit] mediawiki...Popups[master]: actions: Correctly delay FETCH_COMPLETE

2017-04-06 Thread Phuedx (Code Review)
Phuedx has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/346830 ) Change subject: actions: Correctly delay FETCH_COMPLETE .. actions: Correctly delay FETCH_COMPLETE I496fe317 caused a regression where the

  1   2   3   4   5   6   7   8   >