Opened 11 months ago
Closed 11 months ago
#51743 closed task (blessed) (fixed)
Core auto-updates UI: Consider setting it by default for new installations
Reported by: | audrasjb | Owned by: | |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 5.6 |
Component: | Upgrade/Install | Keywords: | has-patch has-screenshots |
Focuses: | Cc: |
Description
During the last dev chat, setting major auto-updates by default for new installations was considered by the core team. The project leads also wanted to explore this for WP 5.6.
Attachments (5)
Change History (23)
#2
@
11 months ago
Patch 51743.diff working fine for me on WordPress 5.6-beta3-49552 with the fresh installation.
This ticket was mentioned in Slack in #core-auto-updates by audrasjb. View the logs.
11 months ago
This ticket was mentioned in Slack in #core-auto-updates by audrasjb. View the logs.
11 months ago
#5
@
11 months ago
- Keywords needs-testing added
In 51743.2.diff: Set the default for the auto_update_core_major
option to enabled
for new installs and override it to unset
on upgrading from a WP version earlier than 5.6.
Please test!
#6
@
11 months ago
In 51743.3.diff:
- Populate the three new auto-update options: auto_update_core_dev, auto_update_core_minor, and auto_update_core_major.
- Add some more inline comments.
These pre-populated options apply only to single site installs despite that they are set/read with update_site_option()
and get_site_option()
. To do the same for multisite/network installs it seems the code will have to be in upgrade_network()
and (perhaps) in populate_network()
in schema.php
but not sure if that's needed/a good idea. Pinging @jeremyfelt as the resident multisite/network expert for a quick review :)
Still todo:
- More testing on existing and new single-site installs.
- TBD: should new network installs be set to do major core auto-updates by default, same as single sites?
This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.
11 months ago
This ticket was mentioned in Slack in #core by helen. View the logs.
11 months ago
This ticket was mentioned in Slack in #core-auto-updates by pbiron. View the logs.
11 months ago
#11
@
11 months ago
Does this ticket need anything else? Do we need to address multisite at all? My view is that by nature multisite is still really meant for a more hands-on admin so I don't know that autoupdates on by default makes sense right now. That said, is it weird if somebody has explicitly turned major autoupdates on in the UI and then converts to multisite and it's off again? Is it okay to call that a multisite conversion quirk for now? I am not strongly opinionated, hoping somebody else feels strongly enough about this to advocate for a direction before RC :)
This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.
11 months ago
This ticket was mentioned in Slack in #core-auto-updates by hellofromtonya. View the logs.
11 months ago
#14
@
11 months ago
- Keywords dev-feedback needs-testing removed
FWIW, I'm fine with the current implementation _for now_. Multisites can be handled in the next release. Also, they still have the possibility to opt-in to auto-updates once the multisite is set up.
#15
@
11 months ago
I concur with @audrasjb.
Ultimately, when a single site is converted to a multisite, it would be good if the multisite "inherited" the current setting for the single site during conversion, but I seriously doubt we can get a working/tested patch before RC1 (in a few hours). I know I certainly don't have time.
This "quirk" should be made very explicit in the dev-note
so that MS admins are aware.
#16
@
11 months ago
This "quirk" should be made very explicit in the dev-note so that MS admins are aware.
Absolutely. FYI I'm working on the dev note right now, I'll let you know when it's ready for review.
Upgrade/Install: set
auto_update_core_major
option totrue
for new installations