Conversation
scryptmouse
left a comment
There was a problem hiding this comment.
I would fall back to the books ordering rather than a dynamic ordering here. The dynamic ordering stuff is intended to be used in cases where there isn't a sufficient ordering. It's less efficient than actual orderings.
| } | ||
| }, | ||
| { | ||
| "path": "book", |
There was a problem hiding this comment.
You have book here but specify featured.books in the template. Are you meaning to select a single book or a group? If a group, rename this to books and use the type entities instead of entity. Otherwise the template needs to be updated.
It looks like we're falling back to dynamic orderings in a number of places in main layouts. Are there others that should be updated to fall back to the named ordering as well? |
No description provided.