- Issue created by @quietone
- Status changed to Postponed
8 months ago 7:42am 11 January 2024 - Status changed to Needs work
7 months ago 11:47am 20 February 2024 - π³π±Netherlands Spokje
So our decision to not remove the deprecated modules related CSS in 11.x bites us here.
core/themes/olivero/config/optional/block.block.olivero_book_navigation.yml
has a dependency on book, which triggers a deprecation in\Drupal\KernelTests\Config\DefaultConfigTest::testThemeConfig
.Not quite sure what to do here, maybe special case this situation in
DefaultConfigTest
, either specific for olivero and books, or better in general? - π¬π§United Kingdom catch
If modules can provide default config for themes, we could move the block over and not the CSS, but... can they even do that??
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Yeah dependencies supports thèmes sonth so that should be doable - nice idea!
- π³π±Netherlands Spokje
Ok, so I moved the dependency and block settings over to a optional config yaml in the book module, that didn't break too much, but I like to be sure this is the correct approach before trying to fix the new test-failures.
- π¬π§United Kingdom catch
From looking at a couple of the test failures they look like ones we'd probably have to deal with when we move book out of core - like comparing default config etc. if that's the case then I think it makes sense to do it this way.
- π³π±Netherlands Spokje
Testfailures where due to me not moving the whole block but trying to be clever.
(Trust me, that never works out...)Straight up move of the
block.block.olivero_book_navigation.yml
to the book module appeases all tests. - Status changed to Needs review
7 months ago 3:46pm 21 February 2024 - π³π±Netherlands Spokje
Ah sorry, thought one was already there.
It wasn't, added one. - Status changed to RTBC
7 months ago 4:41pm 21 February 2024 - πΊπΈUnited States smustgrave
Thanks! Reviewing this like I did Forum and believe everything has been addressed.
- Status changed to Postponed
7 months ago 12:32am 22 February 2024 - π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
We need a stable release of the contrib module before we can proceed here
Thanks
- Assigned to pwolanin
- Status changed to Needs review
6 months ago 10:06pm 1 March 2024 - π«π·France andypost
@pwolanin Now it needs to move code to contrib module according to https://www.drupal.org/about/core/policies/core-change-policies/module-o... β
After that RTBC +1
- Status changed to Needs work
6 months ago 11:37am 12 March 2024 The Needs Review Queue Bot β tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide β to find step-by-step guides for working with issues.
- Status changed to Postponed
6 months ago 3:57pm 25 March 2024 - π¬π§United Kingdom catch
This still needs a stable release of the contrib module per #16 at https://www.drupal.org/project/book β
- πΊπΈUnited States smustgrave
Will do one by 10.3 for sure. Think we were waiting for π Add validation constraints to book.settings Needs review and π Update MigratePluginManager to include both attribute and annotation class Fixed to land to avoid multiple subtree splits.
But if we can do the sub-tree split without those and just backport manually without a full sub-tree split we can do a release today.
- π¬π§United Kingdom catch
π Add validation constraints to book.settings Needs review may have its own dependencies, so I don't think the book subtree split should wait on it. If it had already gone in, it would have been worth waiting for, but it's still got open questions so might not even land in 10.3.x
π Update MigratePluginManager to include both attribute and annotation class Fixed looks a bit closer so might still be worth trying to push that through first.
However we're coming up close to the 10.3 beta deadline, and still need to do module removals and update test fixture changes in the next three to four weeks. I'm not sure if/how many fixtures include book module, but if they do, then it might be better to just go ahead here.
- Status changed to Needs work
5 months ago 12:29am 4 April 2024 - πΊπΈUnited States smustgrave
Migrate one just landed so we can do the sub tree split.
- Status changed to Needs review
5 months ago 2:18pm 4 April 2024 - πΊπΈUnited States smustgrave
A 1.0.0 release has been made for book
Rebased this MR.
- Status changed to RTBC
5 months ago 3:07pm 4 April 2024 - πΊπΈUnited States smustgrave
I really just rebased this. Work was already done so think I'm good to RTBC.
- Status changed to Fixed
5 months ago 3:16pm 4 April 2024 - π¬π§United Kingdom catch
Reviewed this again, all looks straightforwards. Some core modules I am pleased about deprecating and have been trying to get rid of for years, but I still like and use book module so this is a shame, but it's just not used as much as it was, and there are other options in contrib these days too. I'll need to install the contrib book module for at least one site.
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!
- Issue was unassigned.
Automatically closed - issue fixed for 2 weeks with no activity.