Skip to content

Chapter and Book Conversion Actions - #3499

Merged
ssddanbrown merged 11 commits into
developmentfrom
convert_hierachy
Jun 20, 2022
Merged

ssddanbrown merged 11 commits into
developmentfrom
convert_hierachy

Conversation

@ssddanbrown

@ssddanbrown ssddanbrown commented Jun 13, 2022

Copy link
Copy Markdown
Member

This is primarily intended to add actions for converting chapters to books, and books to shelves as per #1087.

Considerations

  • When a book is converted to a shelf, The book will remain if it contains any direct child pages but will have Pages appended to the name and this book will be added to the new "Converted" shelf.
  • These changes additionally bring in availability to set/update book/shelf cover images via the API.

TODO

These are in-addition to the in-code todos.

  • Move added text to translations.
  • Add tests to cover newly available API shelf/book cover image params.
  • We should show a warning in regards to the difference in permissions for shelves when promoting a book.
  • We should shown a warning about handling direct pages when promoting a book.
  • Test setting book cover image from API
    • Ensure we can update without resetting image accidentally
    • Ensure api docs correct.
    • As all above but for shelves.
  • Add tests to cover each require route permission.

- Updates book/shelf cover image handling for easier cloning/handling.
- Adds core logic for promoting books/chapters up a level.
- Enables usage of book/shelf cover image via API.

Related to #1087
- Added required UI within edit view.
- Added required routes and controller actions.
- Also extracted shelf to book view elements to own partial.
- Fixed some existing logic including image param handling in update
  request and activity logging against correct element.
Also updated cloner class with typed properties.
- Updated 'removePermissionFromUser' test helper to work for
  entity-permissions that become part of the joint permissions system.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant