| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add cul_period to cu_log | mediawiki/extensions/CheckUser | master | +311 -0 | |
| Add cul_reason_id and cul_reason_plaintext_id into cu_log | mediawiki/extensions/CheckUser | master | +264 -3 |
Details
Details
| Status | Subtype | Assigned | Task | |
|---|---|---|---|---|
| · · · | ||||
| Open | None | T204347 Duration of query recent changes in Check user tool | ||
| Resolved | Zabe | T233004 Update CheckUser for actor and comment table | ||
| Resolved | Dreamy_Jazz | T309454 cul_reason in cu_log should use the comment table | ||
| Declined | None | T309982 Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_actor to the cu_log table | ||
| · · · |
Event Timeline
Comment Actions
You probably need to make a Schema-change ticket and then once merged, one for production
Comment Actions
I think the above ticket is a schema-change one. The patch was merged but has been reverted. I presume I will need to have the DB changes merged in then before I request it for production.
Comment Actions
Not sure that this is the right status, but want to note that I am working on this and it's not ready yet.
Comment Actions
As https://wikitech.wikimedia.org/wiki/Schema_changes#Workflow_of_a_schema_change states, let's use the Schema-change-in-production once the change has been merged and not before.
Thanks!
Comment Actions
Change 803298 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):
[mediawiki/extensions/CheckUser@master] Add cul_reason_id and cul_reason_plaintext_id into cu_log
Comment Actions
Change 803302 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):
[mediawiki/extensions/CheckUser@master] Add cul_period_id to the cu_log table
Comment Actions
I'm currently working on getting the DB changes merged into the extension. Once they are merged this task will likely include some other changes by Zabe to remove duplication of tasks. All would affect the tables created and used exclusively by the CheckUser extension. The extension has just moved to an abstract schema so this may make this process easier.
Comment Actions
Change 803298 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Add cul_reason_id and cul_reason_plaintext_id into cu_log
Comment Actions
Change 814353 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):
[mediawiki/extensions/CheckUser@master] Add cul_period to cu_log
Comment Actions
Change 814353 abandoned by Dreamy Jazz:
[mediawiki/extensions/CheckUser@master] Add cul_period to cu_log
Reason:
Going with cul_period_id so that any checks made via the API can have their period value logged.