Page MenuHomePhabricator

Update editattemptstep schema documentation for the anonymous_user_token field
Closed, ResolvedPublic

Description

The anonymous_user_token field in the EditAttemptStep schema was added in T221195#5260013. The documentation doesn't reflect that the field is only set for IP editors that are part of the VE mobile default test, per T225209, and therefore might confuse the reader to believe that this field can be used to reliably distinguish between IP editors. That's not the case as the field largely contains NULL-values.

Solution: submit a patch that updates the description of the field in the schema.

Event Timeline

nettrom_WMF added a subscriber: jwang.

Adding @jwang for awareness, but no work is requested.

Change 725351 had a related patch set uploaded (by Nettrom; author: Nettrom):

[schemas/event/secondary@master] Update documentation for anonymous_user_token

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

nettrom_WMF edited subscribers, added: MNeisler; removed: jwang.

Oops, my bad! This isn't a Web team-related task, it's Editing, so I'm updating the list of subscribers.

@MNeisler : This should be ready for reviewing. Could you or one of the engineers on the Editing team review it? Hopefully not a change that requires much efforts since it's updating the documentation.

Change 725351 merged by jenkins-bot:

[schemas/event/secondary@master] Update documentation for anonymous_user_token

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

nettrom_WMF added a subscriber: mpopov.

Patch is merged and documentation updated. Thanks @MNeisler and @mpopov for reviewing and helping out with this patch!