Page MenuHomePhabricator

Mvolz (Marielle Volz)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Oct 15 2014, 9:50 PM (314 w, 5 d)
Availability
Available
IRC Nick
Mvolz
LDAP User
Mvolz
MediaWiki User
Mvolz (WMF) [ Global Accounts ]

mvolz@wikimedia.org

Recent Activity

Sat, Oct 24

Mvolz created T266393: Attempted to edit en page of translatable page causes uncaught TypeError.
Sat, Oct 24, 8:10 PM · VisualEditor, VisualEditor-MediaWiki-2017WikitextEditor

Thu, Oct 22

Mvolz added a comment to T244847: Future of the OpenRefine Wikidata reconciliation interface.

I should probably also point out that earlier versions of the citoid tool on wikidata for wikidata references used open refine and produced better results.

Thu, Oct 22, 1:24 PM · Wikidata, WMSE-Tools-for-Partnerships-2019-Blueprinting, OpenRefine
Mvolz awarded T244847: Future of the OpenRefine Wikidata reconciliation interface a Like token.
Thu, Oct 22, 1:17 PM · Wikidata, WMSE-Tools-for-Partnerships-2019-Blueprinting, OpenRefine

Sat, Oct 17

Mvolz renamed T265354: VisualEditor should add a space before every pipe character used in a cite template from citoid should add a space before every pipe character used in a cite template to VisualEditor should add a space before every pipe character used in a template.
Sat, Oct 17, 2:53 PM · VisualEditor
Mvolz added a comment to T265546: TypeError: undefined is not a constructor (evaluating 'new ve.ui.MWEducationPopupWidget') / TypeError: ve.ui.MWEducationPopupWidget is not a constructor.

This looks like a mismatch between VisualEditor and Citoid - I had this issue locally too on master with Citoid until I updated VisualEditor to master. Some code was moved about between the two.

Sat, Oct 17, 2:12 PM · JavaScript, Citoid, Wikimedia-production-error

Thu, Oct 8

Mvolz updated the task description for T239459: service-runner apps running on kubernetes emit logs with log level 50 .
Thu, Oct 8, 12:22 PM · Patch-For-Review, Platform Engineering (Icebox), CX-cxserver, serviceops-radar, Product-Infrastructure-Team-Backlog, Operations
Mvolz updated the task description for T264083: Switch to WorldCat Search v2 api which uses json.
Thu, Oct 8, 10:00 AM · Technical-Debt, VisualEditor, Citoid
Mvolz added a comment to T160845: wrong author listed and wrong first/last name for the one author listed using ISBN lookup.

Change 632732 had a related patch set uploaded (by Mvolz; owner: Mvolz):
[mediawiki/services/citoid@master] [WIP] Divide authors by first name and last name

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

Thu, Oct 8, 9:58 AM · Patch-For-Review, Citoid, VisualEditor

Tue, Oct 6

Mvolz closed T248571: Update citoid to latest service-template-node as Resolved.
Tue, Oct 6, 8:25 AM · Technical-Debt, Citoid

Fri, Oct 2

Mvolz added a comment to T115224: On URL submission, look up the archived page in the Internet Archive's index and add to the return data.

I would like to emphasize that the availability API has seen improvements since 2017. Is it possible you can redo your investigation a bit?

Fri, Oct 2, 10:40 AM · Internet-Archive, VisualEditor-MediaWiki-References, VisualEditor, Citoid

Wed, Sep 30

Mvolz added a comment to T108713: Change label "Basic form" to something clearer.

I agree with @Amire80 - the issue here is really the word "form". If we were in an universe where the term "template" wasn't such a loaded word, I'd suggest "Basic Template".

Wed, Sep 30, 1:05 PM · Voice & Tone, VisualEditor-MediaWiki-References, Design, Citoid, VisualEditor
Mvolz added a comment to T239459: service-runner apps running on kubernetes emit logs with log level 50 .

The hold-up seems to be eventstreams; it actually uses a fork of service runner, and the fork is missing the feature, so it's slightly more complicated to update. https://github.com/wikimedia/service-runner/tree/prometheus_metrics

Wed, Sep 30, 9:27 AM · Patch-For-Review, Platform Engineering (Icebox), CX-cxserver, serviceops-radar, Product-Infrastructure-Team-Backlog, Operations
Mvolz updated the task description for T239459: service-runner apps running on kubernetes emit logs with log level 50 .
Wed, Sep 30, 9:25 AM · Patch-For-Review, Platform Engineering (Icebox), CX-cxserver, serviceops-radar, Product-Infrastructure-Team-Backlog, Operations

Tue, Sep 29

Mvolz added a project to T264083: Switch to WorldCat Search v2 api which uses json: Technical-Debt.
Tue, Sep 29, 10:40 AM · Technical-Debt, VisualEditor, Citoid
Mvolz added a comment to T160845: wrong author listed and wrong first/last name for the one author listed using ISBN lookup.

It looks like they've released a v2 of their search api since I last looked that returns JSON: https://developer.api.oclc.org/wcv2 (instead of MarcXML or Dublin core, which is what we use now).

I think switching to that is a good idea, although even just using their sample input there's still a bit of junk left over in the authors field (a stray period!) But probably better than doing that processing ourselves!

Tue, Sep 29, 10:39 AM · Patch-For-Review, Citoid, VisualEditor
Mvolz created T264083: Switch to WorldCat Search v2 api which uses json.
Tue, Sep 29, 10:37 AM · Technical-Debt, VisualEditor, Citoid
Mvolz added a comment to T160845: wrong author listed and wrong first/last name for the one author listed using ISBN lookup.

@Mvolz - I would be fine with that, but there are a couple caveats:

  • The refToolbar code isn't foolproof, and in fact, there is no foolproof way to handle the WorldCat author data since its formatting is just too inconsistent.
  • It currently handles "Jr." and "Sr.", but not suffixes or prefixes that may exist in other languages.

If you want to use it, the most recent version of the code is at https://github.com/alexz-enwp/reftoolbar/blob/master/lookup.php#L139.

In the meantime, I'll see if we can ask our contact at WorldCat if there are any better options.

Tue, Sep 29, 10:35 AM · Patch-For-Review, Citoid, VisualEditor

Mon, Sep 28

Mvolz added a comment to T160845: wrong author listed and wrong first/last name for the one author listed using ISBN lookup.

@Whatamidoing-WMF - It looks like the Library of Congress is using the same "weird" punctuation and capitalization as WorldCat:

Maybe it's some kind of library database standard or maybe the LOC is reusing WorldCat data for their own books.
@Mvolz - Can you report the titles that you get for the following ISBN numbers via Zotero's API: 9781608468553, 9780553803549, 9780912647180. Just want to confirm that they are similar to those returned by the LOC search.

Mon, Sep 28, 12:54 PM · Patch-For-Review, Citoid, VisualEditor
Mvolz added a comment to T219919: Move citoid logging to new logging pipeline.

It looks like citoid is now on k8s but still using gelf for logging, possibly the easiest at this point is switching to stdout logging like other node services on k8s. What do you think @Mvolz ?

Mon, Sep 28, 12:42 PM · observability, Citoid, Platform Team Legacy (Watching / External), Services (watching), service-runner, Wikimedia-Logstash, Operations
Mvolz closed T263012: User service-runner provided test server in citoid, a subtask of T219427: Use service-runner provided test server in service-template, as Resolved.
Mon, Sep 28, 12:33 PM · Platform Team Legacy (Later), Services (later), service-template-node
Mvolz closed T263012: User service-runner provided test server in citoid, a subtask of T248571: Update citoid to latest service-template-node, as Resolved.
Mon, Sep 28, 12:33 PM · Technical-Debt, Citoid
Mvolz closed T263012: User service-runner provided test server in citoid as Resolved.
Mon, Sep 28, 12:33 PM · Platform Team Legacy (Later), Services (later), service-template-node

Sep 25 2020

Mvolz awarded Blog Post: CI now updates your deployment-charts a Party Time token.
Sep 25 2020, 8:57 AM

Sep 24 2020

Mvolz added a comment to T263632: Citoid generate different citation in arwiki compared with enwiki.

It's likely a difference between Cite book on en wiki and the template that is used on ar wiki for books?

Sep 24 2020, 10:27 AM · Citoid

Sep 23 2020

Mvolz closed T208305: Support 'maps' parameter in TemplateData Editor (GUI) as Resolved.
Sep 23 2020, 9:58 AM · User-Ryasmeen, Google-Summer-of-Code (2020), VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz closed T208305: Support 'maps' parameter in TemplateData Editor (GUI), a subtask of T207139: Deprecate manual raw TemplateData JSON source editing, as Resolved.
Sep 23 2020, 9:57 AM · VisualEditor, TemplateData

Sep 16 2020

Mvolz closed T262799: Citoid does not recognize DOI as Invalid.
Sep 16 2020, 11:55 AM · Citoid
Mvolz added a comment to T262799: Citoid does not recognize DOI.

I'm unable to replicate this, it looks like it works: https://en.wikipedia.org/api/rest_v1/data/citation/mediawiki/10.1098%2Frsta.2014.0376

Sep 16 2020, 11:02 AM · Citoid
Mvolz added a comment to T261875: Test instance: TemplateData editor re-order sections.

Just FYI but the reason we put maps above parameters is because there are often a lot of parameters, which means that in many cases the edit maps visible button isn't visible because it's "below the fold." This was an intentional design decision we made.

Sep 16 2020, 10:26 AM · Patch-For-Review, Unplanned-Sprint-Work, WMDE-QWERTY-Sprint-2020-08-26, WMDE-Templates-FocusArea
Mvolz renamed T263012: User service-runner provided test server in citoid from User service runner provided test server in citoid to User service-runner provided test server in citoid.
Sep 16 2020, 10:14 AM · Platform Team Legacy (Later), Services (later), service-template-node
Mvolz added a subtask for T248571: Update citoid to latest service-template-node: T263012: User service-runner provided test server in citoid.
Sep 16 2020, 10:06 AM · Technical-Debt, Citoid
Mvolz added a parent task for T263012: User service-runner provided test server in citoid: T248571: Update citoid to latest service-template-node.
Sep 16 2020, 10:06 AM · Platform Team Legacy (Later), Services (later), service-template-node
Mvolz claimed T263012: User service-runner provided test server in citoid.
Sep 16 2020, 10:06 AM · Platform Team Legacy (Later), Services (later), service-template-node
Mvolz created T263012: User service-runner provided test server in citoid.
Sep 16 2020, 10:03 AM · Platform Team Legacy (Later), Services (later), service-template-node

Sep 9 2020

Mvolz added a comment to T258572: Refactor our helmfile.d dir structure for services.

I'm planning to deploy tomorrow, so I was wondering if I can have clarification on https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/622585

The eqiad file was renamed to just values, but I noticed in the examples folder there's both values.yaml and values-eqiad.yaml

So when I do a deploy, will I be able to deploy to eqiad, or is this a mistake and there should also be a separate eqiad file in this change in order to deploy to eqiad?

You're totally able to deploy to eqiad without a values-eqiad.yaml. Those environment specific values files can be used to overwrite settings (for that environment) but are not required. Please see https://gerrit.wikimedia.org/r/plugins/gitiles/operations/deployment-charts/+/refs/heads/master/helmfile.d/services/README.MD for a description on which files are applied in which order (please tell if anything is unclear or missing! :))

Sep 9 2020, 12:18 PM · Patch-For-Review, Prod-Kubernetes, Release Pipeline, serviceops
Mvolz added a comment to T258572: Refactor our helmfile.d dir structure for services.

I'm planning to deploy tomorrow, so I was wondering if I can have clarification on https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/622585

Sep 9 2020, 12:10 PM · Patch-For-Review, Prod-Kubernetes, Release Pipeline, serviceops
Mvolz added a comment to T255868: Move citoid to use TLS only.
Sep 9 2020, 12:09 PM · Citoid, Patch-For-Review, Prod-Kubernetes, Kubernetes, serviceops, Operations
Mvolz added a project to T255868: Move citoid to use TLS only: Citoid.
Sep 9 2020, 11:54 AM · Citoid, Patch-For-Review, Prod-Kubernetes, Kubernetes, serviceops, Operations
Mvolz updated the task description for T262266: Retarget TemplateData extension for Parsoid .
Sep 9 2020, 11:51 AM · Editing-team (Tracking), TemplateData, Parsoid-Rendering, Parsoid, VisualEditor

Sep 8 2020

Mvolz triaged T262266: Retarget TemplateData extension for Parsoid as Medium priority.
Sep 8 2020, 12:29 PM · Editing-team (Tracking), TemplateData, Parsoid-Rendering, Parsoid, VisualEditor
Mvolz created T262266: Retarget TemplateData extension for Parsoid .
Sep 8 2020, 12:18 PM · Editing-team (Tracking), TemplateData, Parsoid-Rendering, Parsoid, VisualEditor
Mvolz updated the task description for T250292: Change the value of 'templatedata-doc-param-type-wiki-page-name' from "Page" to "Page name" in English (en.json file only).
Sep 8 2020, 12:17 PM · Patch-For-Review, good first task, VisualEditor, TemplateData, I18n
Mvolz renamed T250292: Change the value of 'templatedata-doc-param-type-wiki-page-name' from "Page" to "Page name" in English (en.json file only) from [[MediaWiki:Templatedata-doc-param-type-wiki-page-name/da]] translation/naming issue to Change the value of 'templatedata-doc-param-type-wiki-page-name' from "Page" to "Page name" in English (en.json file only).
Sep 8 2020, 12:16 PM · Patch-For-Review, good first task, VisualEditor, TemplateData, I18n
Mvolz updated the task description for T248571: Update citoid to latest service-template-node.
Sep 8 2020, 8:37 AM · Technical-Debt, Citoid

Sep 7 2020

Mvolz updated subscribers of T211411: Citoid automated monitoring times out due to Zotero v2.

There have been no timeouts recorded by the automatic check scripts since the deploy, so looking good. Resolving for now, let's reopen if things change.

Sep 7 2020, 9:21 AM · serviceops, Prod-Kubernetes, Platform Team Legacy (Watching / External), Services (watching), Citoid, Operations

Sep 6 2020

alanajjar awarded T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia a Love token.
Sep 6 2020, 9:58 AM · Citoid
Mvolz added a comment to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia.

I've reverted these two changes with my work account Mvolz (WMF), but another user will need to approve the change since it's pending review.

Thanks @Mvolz. I approved all of them. And I can confirmed that Automatic cite works again. (I'll ask users with complaints to try it, then we can close this task).

What about the error message not substituted ?

citoid-citefromiddialog-mode-manual is the wrong message, I think... we refractored it. T219512. Looks like it didn't get retranslated with the right message name?

Yup, this should be citoid-citoiddialog-mode-manual instead.

https://translatewiki.net/w/i.php?title=Special:Translate&showMessage=citoid-citoiddialog-use-general-error-message&group=ext-citoid&language=ar&filter=&optional=1&action=translate needs to be fixed to use the correct dialog name, if you have an account on translatewiki?

Sep 6 2020, 9:06 AM · Citoid
Mvolz added a project to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia: translatewiki.net.
Sep 6 2020, 9:01 AM · Citoid
Mvolz added a comment to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia.

I've reverted these two changes with my work account Mvolz (WMF), but another user will need to approve the change since it's pending review.

Thanks @Mvolz. I approved all of them. And I can confirmed that Automatic cite works again. (I'll ask users with complaints to try it, then we can close this task).

What about the error message not substituted ?

citoid-citefromiddialog-mode-manual is the wrong message, I think... we refractored it. T219512. Looks like it didn't get retranslated with the right message name?

Sep 6 2020, 9:01 AM · Citoid
Mvolz added a comment to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia.

I've reverted these two changes with my work account Mvolz (WMF), but another user will need to approve the change since it's pending review.

Thanks @Mvolz. I approved all of them. And I can confirmed that Automatic cite works again. (I'll ask users with complaints to try it, then we can close this task).

What about the error message not substituted ?

Sep 6 2020, 8:57 AM · Citoid
Mvolz added a comment to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia.

If you look at https://ar.m.wikipedia.org/wiki/%D8%AE%D8%A7%D8%B5:%D8%AA%D8%A7%D8%B1%D9%8A%D8%AE/%D9%82%D8%A7%D9%84%D8%A8:%D8%A7%D8%B3%D8%AA%D8%B4%D9%87%D8%A7%D8%AF_%D9%88%D9%8A%D8%A8/%D8%B4%D8%B1%D8%AD someone has deleted all the template data. That needs to be reverted on all the templates where it has been done as citoid will not work without template data.

Sep 6 2020, 8:27 AM · Citoid

Sep 5 2020

Mvolz added a comment to T262129: Citoid says "We couldn't make a citation for you" for some sites on Arabic Wikipedia.

If you look at https://ar.m.wikipedia.org/wiki/%D8%AE%D8%A7%D8%B5:%D8%AA%D8%A7%D8%B1%D9%8A%D8%AE/%D9%82%D8%A7%D9%84%D8%A8:%D8%A7%D8%B3%D8%AA%D8%B4%D9%87%D8%A7%D8%AF_%D9%88%D9%8A%D8%A8/%D8%B4%D8%B1%D8%AD someone has deleted all the template data. That needs to be reverted on all the templates where it has been done as citoid will not work without template data.

Sep 5 2020, 8:27 PM · Citoid

Sep 3 2020

Mvolz updated the task description for T248571: Update citoid to latest service-template-node.
Sep 3 2020, 7:27 PM · Technical-Debt, Citoid
Mvolz updated the task description for T248571: Update citoid to latest service-template-node.
Sep 3 2020, 10:55 AM · Technical-Debt, Citoid
Mvolz updated the task description for T248571: Update citoid to latest service-template-node.
Sep 3 2020, 10:55 AM · Technical-Debt, Citoid
Mvolz created T261941: Model adding / removing individual maps in Model.js and test methods..
Sep 3 2020, 10:22 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz closed T261406: Add maps to QUnit tests as Resolved.
Sep 3 2020, 10:20 AM · User-Ryasmeen, MW-1.36-notes (1.36.0-wmf.8; 2020-09-08), Patch-For-Review, Google-Summer-of-Code (2020), VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz closed T261406: Add maps to QUnit tests, a subtask of T208305: Support 'maps' parameter in TemplateData Editor (GUI), as Resolved.
Sep 3 2020, 10:20 AM · User-Ryasmeen, Google-Summer-of-Code (2020), VisualEditor, Outreach-Programs-Projects, TemplateData

Aug 27 2020

Mvolz added a comment to T259928: Gale URLs don't produce good Citations via Citoid.

Yep, I was using the getlink urls like:
*https://link.gale.com/apps/doc/A171034250/GPS?u=wikipedia&sid=GPS&xid=dbbf72df
https://link.gale.com/apps/doc/A171034250/GPS?u=wikipedia&sid=GPS&xid=dbbf72df
, but they weren't generating citations on Citoid so fell back on the
preformatted citations which are three buttons over. *

Aug 27 2020, 1:06 PM · Citoid, The-Wikipedia-Library
Mvolz updated the task description for T261406: Add maps to QUnit tests.
Aug 27 2020, 12:56 PM · User-Ryasmeen, MW-1.36-notes (1.36.0-wmf.8; 2020-09-08), Patch-For-Review, Google-Summer-of-Code (2020), VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz created T261406: Add maps to QUnit tests.
Aug 27 2020, 12:46 PM · User-Ryasmeen, MW-1.36-notes (1.36.0-wmf.8; 2020-09-08), Patch-For-Review, Google-Summer-of-Code (2020), VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz added a comment to T115224: On URL submission, look up the archived page in the Internet Archive's index and add to the return data.

Declined because I think the adding of archive links is better left to bots.

Aug 27 2020, 11:07 AM · Internet-Archive, VisualEditor-MediaWiki-References, VisualEditor, Citoid
Mvolz closed T115224: On URL submission, look up the archived page in the Internet Archive's index and add to the return data as Declined.
Aug 27 2020, 11:05 AM · Internet-Archive, VisualEditor-MediaWiki-References, VisualEditor, Citoid
Mvolz closed T115224: On URL submission, look up the archived page in the Internet Archive's index and add to the return data, a subtask of T95388: Try to find link in archive.org for 520 page, as Declined.
Aug 27 2020, 11:05 AM · Internet-Archive, VisualEditor, Citoid
Mvolz closed T255176: Citoid returns mangled results for Washington Post articles as Resolved.
Aug 27 2020, 11:05 AM · Citoid
Mvolz added a comment to T115224: On URL submission, look up the archived page in the Internet Archive's index and add to the return data.

Just an FYI, but I investigated adding this with a WIP patch here (a few years ago): https://gerrit.wikimedia.org/r/c/mediawiki/services/citoid/+/375810

Aug 27 2020, 10:54 AM · Internet-Archive, VisualEditor-MediaWiki-References, VisualEditor, Citoid

Aug 24 2020

Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 24 2020, 10:14 AM · VisualEditor, Outreach-Programs-Projects, TemplateData

Aug 21 2020

Mvolz updated subscribers of T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).
Aug 21 2020, 11:04 AM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz moved T260980: Validate parameters in maps to ensure they're valid template data parameters. from Backlog to Next-up on the TemplateData board.
Aug 21 2020, 11:03 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:44 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:35 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:34 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:34 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:33 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz updated the task description for T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:32 AM · VisualEditor, Outreach-Programs-Projects, TemplateData
Mvolz created T260980: Validate parameters in maps to ensure they're valid template data parameters. .
Aug 21 2020, 10:30 AM · VisualEditor, Outreach-Programs-Projects, TemplateData

Aug 11 2020

Mvolz added a comment to T255176: Citoid returns mangled results for Washington Post articles.

@Mvolz - Now that the code is merged locally, what happens next? Does someone need to deploy the new version to the production servers?

Aug 11 2020, 6:55 PM · Citoid

Aug 10 2020

Mvolz created T260054: is:wip flag doesn't seem to be working in gerrit with reviewers .
Aug 10 2020, 1:19 PM · Gerrit

Aug 8 2020

Akuckartz awarded T107693: Add "Create new item" link to property value drop-down menu (particularly when no item matches) in Wikidata a Like token.
Aug 8 2020, 12:44 PM · Google-Code-in-2017, Wikidata-Gadgets, MediaWiki-extensions-WikibaseRepository, Wikidata

Aug 6 2020

Mvolz added a comment to T257809: Automatic citation on Visual Editor doesnt work at trwiki.

I couldn't find out what the problem was caused by. I would appreciate it if you review it at a convenient time.

Aug 6 2020, 11:06 AM · Citoid, Turkish-Sites

Aug 5 2020

Mvolz added a comment to T258158: Bump zotero base images from stretch to buster.

Requested 10 get added to CI here upstream: https://github.com/zotero/translation-server/issues/118

Aug 5 2020, 12:30 PM · Patch-For-Review, serviceops-radar, Citoid

Aug 4 2020

Mvolz added a comment to T160845: wrong author listed and wrong first/last name for the one author listed using ISBN lookup.

I've now deployed a fix that no longer splits the authors, and just puts them all in the "last" field. So in terms of how it's rendered with most citation templates, it looks a lot better. However this is a bit hacky, so I'm leaving this open for right now. It will continue to leave in parts of the author name that we don't necessarily want, like the birth date in parens.

Unfortunately, this entirely broke all author data for ISBN look-ups for RefToolbar, which excludes any author data that has a comma in it (under the assumption that this indicates the data is malformed). It took me a long time to track down where this regression came from. In the future, could you please post a notice at https://en.wikipedia.org/wiki/Wikipedia_talk:RefToolbar any time you make a significant change to what Citoid returns? Thanks!

Aug 4 2020, 11:53 AM · Patch-For-Review, Citoid, VisualEditor
Mvolz updated subscribers of T259040: Bind Citoid service to a static IP address (or addresses).

@akosiaris is this feasible/desirable?

Aug 4 2020, 11:43 AM · Operations, Services, Citoid

Aug 3 2020

Mvolz renamed T259469: citoid /api LVS check reports HTTP 200 instead of HTTP 404 from citoid /api LVS check reports HTTP 404 instead of HTTP 200 to citoid /api LVS check reports HTTP 200 instead of HTTP 404.
Aug 3 2020, 4:13 PM · Patch-For-Review, Citoid, Operations, serviceops
Mvolz added a comment to T259469: citoid /api LVS check reports HTTP 200 instead of HTTP 404.

@Mvolz any ideas?

Aug 3 2020, 4:09 PM · Patch-For-Review, Citoid, Operations, serviceops
Mvolz added a comment to T91646: Create a process for doing regular citoid service upgrade deployments.

I think this is as resolved as it's going to get - it's been pretty much "as needed" and now we have a window.

Aug 3 2020, 12:44 PM · Platform Team Legacy (Later), Services (next), Citoid
Mvolz closed T91646: Create a process for doing regular citoid service upgrade deployments as Resolved.
Aug 3 2020, 12:43 PM · Platform Team Legacy (Later), Services (next), Citoid

Jul 29 2020

Mvolz added a comment to T257809: Automatic citation on Visual Editor doesnt work at trwiki.

I thought it was working but there is a problem. Some links work, some don't.

For example it works when I use this link: https://www.hurriyet.com.tr/kelebek/keyif/tiyatroya-adanmis-50-yil-tuncer-cucenoglu-41566796
But it doesn't work when I use it: https://www.nytimes.com/2020/07/28/science/nasa-jezero-perseverance.html

What did I do wrong?

Jul 29 2020, 1:34 PM · Citoid, Turkish-Sites

Jul 26 2020

Mvolz added a comment to T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).

https://doc.wikimedia.org/mediawiki-core/master/php/classStatusValue.html#a173df704be03a150626c89412f14fe04 looks like newFatal is a StatusValue fcn now? Maybe things just need to be updated?

Jul 26 2020, 6:13 AM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz added a comment to T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).

I previously thought this was a TemplateData extension issue, but given that this is handled correctly by the 2017 wikitext editor, it might be a core problem? Not sure I tagged the right projects.

Jul 26 2020, 6:10 AM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz renamed T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData) from TemplateData no longer displays errors in old wikitext editor when there are any kind of errors in templatedata on save; instead no error message is displayed but it won't save. to Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).
Jul 26 2020, 6:08 AM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor

Jul 24 2020

Mvolz added a comment to T257809: Automatic citation on Visual Editor doesnt work at trwiki.

Despite all the mishaps, I finally save it. Can you verify that I'm doing it right?

https://tr.wikipedia.org/w/index.php?title=Şablon%3AKaynak%2Fbelge&type=revision&diff=22912043&oldid=17915365

And I also have to do the same to other templates from now on. Isn't it?

Jul 24 2020, 2:29 PM · Citoid, Turkish-Sites
Mvolz added a comment to T257809: Automatic citation on Visual Editor doesnt work at trwiki.

I have activated VisualEditor by adding ?veaction=edit to the end of the URL and deleted a few parameters that gave the error (like doi parameter) However, noticed some working paramaters also gave errors (like ad2). I'm sure the "ad2" parameter is working.

Error: "maps.citoid.podcaster" özelliği için geçersiz "ad2" parametresi.
Translation: Invalid "ad2" parameter for "maps.citoid.podcaster" feature.

Is this a bug too or something else?

Jul 24 2020, 1:42 PM · Citoid, Turkish-Sites
Mvolz renamed T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData) from TemplateData no longer displays errors in old wikitext editor when there are errors in 'maps' values on save; instead no error message is displayed but it won't save. to TemplateData no longer displays errors in old wikitext editor when there are any kind of errors in templatedata on save; instead no error message is displayed but it won't save. .
Jul 24 2020, 1:31 PM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz renamed T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData) from TemplateData no longer displays errors when there are errors in 'maps' values on save; instead no error message is displayed but it won't save. to TemplateData no longer displays errors in old wikitext editor when there are errors in 'maps' values on save; instead no error message is displayed but it won't save. .
Jul 24 2020, 1:03 PM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz added a comment to T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).

This doesn't appear to be a problem in the 2017 wikitext editor, only the old wikitext editor. Unfortunately on some wikis these aren't enabled on Template pages (i.e. where the bug was first detected, Turkish Wikipedia: https://phabricator.wikimedia.org/T257809#6332885 )

Jul 24 2020, 1:02 PM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, Regression, VisualEditor
Mvolz added a comment to T257809: Automatic citation on Visual Editor doesnt work at trwiki.

I was trying to create my first map for the Kaynak template, but I cannot save the page. I review the code and found no errors. The code is here: https://pastebin.com/7uQvyj3x

Could you tell me why I couldn't save and what I did wrong? Thanks.

Jul 24 2020, 12:43 PM · Citoid, Turkish-Sites
Mvolz triaged T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData) as High priority.
Jul 24 2020, 12:42 PM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, VisualEditor, Regression
Mvolz created T258790: Old wikitext editor no longer displays error message when :Status::newFatal() is called (in TemplateData).
Jul 24 2020, 12:42 PM · Platform Team Workboards (Clinic Duty Team), Editing-team (Tracking), MediaWiki-Core-Hooks, TemplateData, VisualEditor, Regression

Jul 23 2020

Mvolz added a comment to T252938: citoid/zotero creates citations without title for makorrishon.co.il, israelhayom.co.il, and news.walla.co.il.

It's possible we might also be getting blocked or something because when I first ran it I was able to get the metadata, but the second time not. It's weird though, When I try to get the html of these locally, it's just a bunch of gibberish - basically a <script> tag in between an html tag and that's it.

Jul 23 2020, 1:32 PM · Citoid