Page MenuHomePhabricator

Change lu_attached_method to not be an enum
Open, Needs TriagePublic

Description

It allows us to easier add new methods. Also due to T119173: RFC: Discourage use of MySQL's ENUM type we likely need to do this at some point anyway.

Event Timeline

Change 773903 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/CentralAuth@master] Add gu_attachment_method

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

Bugreporter added a subscriber: Bugreporter.

Alternatively we can somehow deprecate and remove this field completely.

Alternatively we can somehow deprecate and remove this field completely.

We could probably also just remove it. I don't think it's really used.

Change 773903 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] Add lu_attachment_method column to localuser table

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

Change 776291 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/CentralAuth@master] Add lu_attachment_method migration code

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

Change 804694 had a related patch set uploaded (by Zabe; author: Zabe):

[operations/puppet@production] maintain-views.yaml: Allow selecting lu_attachment_method

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

Change 804694 merged by Ladsgroup:

[operations/puppet@production] maintain-views.yaml: Allow selecting lu_attachment_method

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