- 🇳🇿New Zealand quietone
This extension is being deprecated, see 🌱 [Meta] Tasks to deprecate Book module Active . It will be removed from core and moved to a contrib project, 📌 [11.x] [Meta] Tasks to remove Book Active .
This is now Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project → and the Extensions approved for removal → policies.
This issue may be re-opened if it can be considered critical → , If unsure, re-open the issue and ask in a comment.
- 🇫🇷France liliplanet Cannes
That is just amazing! Thank you for the patch quietone and everyone that contributed.
- Status changed to Needs work
7 months ago 5:34pm 26 April 2024 - 🇺🇸United States mortona2k Seattle
This patch is doing the same thing as https://www.drupal.org/project/book/issues/3183232 ✨ Render book navigation as a pseudo-field to change it's position in node Needs work
Minor differences in book_entity_extra_field_info() label and description.
Close this one?
- 🇺🇸United States DamienMcKenna NH, USA
This needs to be reworked as a MR for v2.
- Status changed to Needs review
2 months ago 11:35am 17 September 2024 - Status changed to Needs work
2 months ago 5:50pm 19 September 2024 - 🇺🇸United States mortona2k Seattle
This is working.
With the patch, I see Book Navigation in manage display, and moving it around or disabling is working.
One issue I ran into is that if you enable a content type to be added to the book, we need to clear the cache to rerun book_entity_extra_field_info().
Seems like that could be done on submission of the book settings config form.
Reviewed on drupal 10.3 setup with standard install profile.
Testing steps:
1. Ran drush updb.
2. New book navigation field appeared in default and teaser display modes.
3. Moved around book navigation field, worked as expected.
4. Enabled layout builder.
5. Moved around book navigation field on a book node. Worked as expected.Moving to RTBC.
- 🇺🇸United States DamienMcKenna NH, USA
Happy to see this committed, thank you everyone!
That said, do we need a update script to force a cache clear, that way the new hook can be triggered so that the extras field(s) are made visible on appropriate view modes?
It might also be worth enabling the field on the "full" view mode for enabled content types and disabled on other view modes.
- 🇺🇸United States smustgrave
@damienmckenna normally yes, but there are currently about 3 update hooks already in the next alpha release. For the second part about the view mode made a follow up?
Planning a few more release before marking a stable one.
- 🇺🇸United States DamienMcKenna NH, USA
Will do, thank you for the follow-up!
Automatically closed - issue fixed for 2 weeks with no activity.