Background
The 'ipinfo-enable' preference is an unnecessary extra step that is causing bugs such as T309365 and T302398.
For now, IPInfo could be considered enabled if the BetaFeature is enabled, and disabled if the BetaFeature is disabled. (The user must still agree to the terms of use before using IPInfo.)
Once IPInfo is no longer a beta feature, we should re-introduce the preference for enabling it.
Acceptance criteria
- remove "enable IPInfo" checkbox from Special:Preferences main page
Testing notes
The following should be tested...
Basic functioning:
- When the beta feature is enabled and the agreement is accepted, IPInfo is visible on Special:Contributions and Special:Log
- When the beta feature is enabled but not accepted, the agreement checkbox appears on Special:Contributions and IPInfo icons do not appear on Special:Log
- When the beta feature is disabled, the IPInfo infobox does not appear on Special:Contributions and IPInfo icons do not appear on Special:Log. The agreement checkbox does not appear on Special:Preferences#mw-prefsection-personal
Logging:
- When the beta feature is first enabled (and the agreement is never accepted at this stage), nothing is logged (whether enabling directly or via checking auto-enroll)
- When the agreement is accepted, 'enable_ipinfo' is logged
- When the agreement is un-accepted, 'disable_ipinfo' is logged
- If IPInfo is already enabled, then when the beta feature disabled, 'disable_ipinfo' is logged
- If IPInfo is already enabled, then when the preferences are reset via Special:Preferences/reset, 'disable_ipinfo' is logged
Notes
The following documentation may be useful: