It's time to branch Sector 9.2. From this point forward Sector 9.1 will only receive security updates, and Sector 5.x (D8) will be marked as deprecated.

Tasks

Media UX Revisions

Contrib Audit

We ran a module audit and decided the following modules should be removed from the distribution and added on a site by site basis.

  • Backup & Migrate
  • Entity Browser
  • Media Entity Browser
  • Google Analytics

Recipe changes under consideration:

We should also do a full contrib module sweep to their latest versions.

Front-End Tasks

Bug Fixes

CommentFileSizeAuthor
#10 sector-meta_9_2_modules-3213516-10.patch904 bytesdieuwe

Comments

DanielVeza created an issue. See original summary.

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Title: [META] - Remove Backup Migrate from Sector » [META] - Remove disabled modules from Sector
Version: 9.1.1 » 9.1.x-dev
Issue summary: View changes

Agreed, and I've noticed that entity_browser is included as disabled by default too, as Media Library took its place long ago.

We should review the full list of modules for anything else that is disabled.

9.2 is perhaps also a good time to review #3189444: Replace honeypot and reCAPTCHA with antibot?, even if just to ship with antibot as a default for new builds and keep honeypot/recaptcha to give existing sites time to switch over before we drop them completely.

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Issue tags: +9.2 Target

Full List:

Backup & Migrate
Entity Browser
Media Entity Browser
Google Analytics

Any objections to this list?

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented

Ah we also have this #3199231: Investigate the removal of Entity Embed.

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Title: [META] - Remove disabled modules from Sector » [META] - Sector 9.2 Roadmap
Issue summary: View changes
danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented

I've just taken over this issue as a generic 9.2 Roadmap.

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Issue summary: View changes

  • dieuwe committed 639b137 on 9.2.x 
    Issue #3213516: [META] - Sector 9.2 Roadmap
dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Version: 9.1.x-dev » 9.2.x-dev
Issue summary: View changes

I've just committed the drupal/core-recommended version change to kickstart the process of getting a 9.2.0 stable release out.

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Status: Active » Needs review
StatusFileSize
new sector-meta_9_2_modules-3213516-10.patch904 bytes

I'm adding a patch to remove optional contrib from the composer.json. None of these modules are installed by default (Backup & Migrate, Entity Browser, Media Entity Browser, Google Analytics) and so should be very safe to remove - sites that have since enabled them can simply update their own composer.json file and will be all good to go.

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Issue summary: View changes

  • DanielVeza committed da03405 on 9.2.x authored by dieuwe 
    Issue #3213516 by dieuwe, DanielVeza: [META] - Sector 9.2 - Remove...
danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented

Committed the disabled modules patch

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Issue summary: View changes

Added the catch-all front end task

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Issue summary: View changes
danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Issue summary: View changes

I've moved a whole bunch of the tasks to 9.3 target

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Issue summary: View changes
Status: Needs review » Needs work

Great, I don't think we have any more blockers identified. We just need full regression testing and release notes.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Related issues: +#3232283: Install error: Class "Drupal\sector\SectorInstallHelpers" does not exist.

Please check #3232283: Install error: Class "Drupal\sector\SectorInstallHelpers" does not exist. - there might be a missing piece. Or I'm missing something, which is *quite* possible on a Friday night ;)

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Issue summary: View changes
Related issues: +#3232290: Single checkbox non-functional in Sector Radix Starter theme, +#3225273: WYSIWYG switching 'source modes' adds erroneous <p> tags each time., +#3232290: Single checkbox non-functional in Sector Radix Starter theme, +#3231653: 9.2 front end fixes, +#3189444: Replace honeypot and reCAPTCHA with antibot?, +#3199231: Investigate the removal of Entity Embed., +#3181009: Media hyperlink renders 'filename' instead of 'title' in default media view mode, +#3228846: Publish media by default, +#3231664: Final (Hopefully) 9.2 core and security updates, +#3229334: Sector README.md is out of date

Adding #3232290: Single checkbox non-functional in Sector Radix Starter theme for consideration too.

dieuwe’s picture

dieuwe
Location Auckland, NZ
commented
Status: Needs work » Fixed

Sector 9.2.0 has been tagged.

Marking this as fixed and any new issues arising will be addressed separately. Thanks for the work on this everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.