Skip to content

Fix KeyError: 'slug' #31

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

Fix KeyError: 'slug' #31

wants to merge 2 commits into from

Conversation

Snaptraks
Copy link

This happens when a book has chapters and not just pages. Fixes #30

This modification works for my specific Bookstack instance but I would greatly appreciate more testing on different setups.

Snaptraks and others added 2 commits December 11, 2023 11:34
This happens when a book has chapters
and not just pages
@pchang388
Copy link
Contributor

hey @Snaptraks thanks for the issue/pr, let me take a look at this when I get some time in a day or two!

@pchang388
Copy link
Contributor

pchang388 commented Dec 19, 2023

Closed in favor of #32 but thank you again for the PR this.

This PR directly led to #32

@pchang388 pchang388 closed this Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

get_child_nodes("pages", ...) tries to load chapters as pages
2 participants