Assigning to Cormac for now so he can assess.

Assigning to Cormac for now so he can assess.

Additional info:

There are two goals for the call to action in UploadWizard, one main one and one "stretch"

Tested on production! Works!

Heads-up - we're going to start addressing some of these issues soon, but there's a refactoring patch that has to go out first (https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/WikibaseMediaInfo/+/528825/).

We'll add a check to the captions field that ensures it's at least 5 characters long (the same rule as Descriptions has)

In T234250#5535274, @Mholloway wrote:

@Ramsey-WMF Where did we come down on excluding images with existing depicts statements? IIRC, we were leaning toward including them anyway.

@MarkTraceur @matthiasmullie - per discussion in backlog grooming today will research increasing the edit rate limits on Commons to reduce the odds of this error happening

@Mholloway thoughts? Beta is pretty borked right now 😿

Hi @EBernhardson Did you intend that patch for this task, or T231038 ?

Tested on production with multiple protected files using multiple transclusions. All good.

Assigning to Edward for testing. This should be available to test (as a qualifier only) on Test Commons tomorrow (Tuesday, Sept 10th)

Confirmed working on production, both mobile and desktop.

Tested on production (on mobile too!). Works.

Tested. Fixed.

As discussed in today's backlog grooming, if UW works, then the refactor worked :) Closing.

Works!

Note: once admins update the page that controls the order of statements, page caches will still be there so we'll either have to PURGE or wait for the caches to clear/expire (default 30 days).

This is an older ticket for just covering the basic functionality of depicts in campaigns. It is done.

Tested on Prod and works (plus Eric and Matthias swear it's fixed).

We may revisit this functionality later, but for now this works.

whoops. mistakenly closed this one

Confirmed. But this is weird because I'm pretty sure I tested this exact scenario at some point during dev and it worked as expected. Regardless, it's certainly a bug now.

Works on prod too. Can close it.

This hit production Commons today and seems to be working. Going to call it done!

We're no longer the Multimedia team (we're "Structured Data" now 😺 )

In T231032#5454811, @Mholloway wrote:

I can add a machine-aided depicts tag (or similar) to every revision adding a depicts statement, as a start.

I have tried to reproduce the bug on Beta for a while now, and I haven't seen it! Good sign that it's no longer an issue 👍🏼

Ah yes, I might have forgotten to uncheck the title checkbox on a copy on the describe step. I'm not entirely sure if I did, but I just tried again and could not reproduce it, and if you can't either I'd say just forget it. I probably borked something on my side 😅

Okay, so here are the results of my latest round of testing on labs:

It doesn't quite work for quantity fields. It seems to work if you start with the top item, but when I started with the 2nd of 3 uploads and set two quantity statements, when I did the copy the statements appeared on the 1st and 3rd files, but the value wasn't set (this seemed to only happen the first time; as I did more tests with the same files, I couldn't replicate the initial problem).

Oh, forgot to add that this seems to be intermittent. I did a lot of UW testing and it mostly worked fine. Every now and then something weird like this would happen though.

Yes, on Beta.

Something I've seen on UW today (autosuggest box getting stuck in "loading" mode), CORS JavaScript errors in the console:

Currently not working on Beta