Page MenuHomePhabricator

[MEX] Remove debug elements
Closed, ResolvedPublic

Description

Remove any debug log or template elements that may have been introduced to support development.

Acceptance Criteria

  • Templates contain no hidden debug elements (in particular, wikibase.mobileUi.statementView.vue .statement_data_debug)
  • Any Javascript console logging introduced is removed, or replaced with mw.log at an appropriate log level (debug or less)

Event Timeline

Change #1176232 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):

[mediawiki/extensions/Wikibase@master] Remove debug element from StatementDetailView

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

Change #1176232 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Remove debug element from StatementDetailView

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

Any Javascript console logging introduced is removed, or replaced with mw.log at an appropriate log level (debug or less)

FWIW, I don’t see any console.log() calls in our code already.

Change #1182600 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] Remove obsolete TODO comment

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

Change #1182600 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Remove obsolete TODO comment

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