Skip to content
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

Fetch all Kolibri content nodes #124

Merged
merged 2 commits into from
Nov 9, 2023
Merged

Fetch all Kolibri content nodes #124

merged 2 commits into from
Nov 9, 2023

Conversation

dbnicholson
Copy link
Member

Kolibri defaults to only downloading content nodes that it believes the current instance can render. Both the image builder and the content server have Kolibri instances that don't necessarily have all available content renderers. To be safe they should download content nodes without regard for whether Kolibri thinks they're renderable. For the importcontent CLI, that's the --include-unrenderable-content option. For the task API, that's the renderable_only option.

Helps: https://github.com/endlessm/endless-key-content-private/issues/103

By default, Kolibri filters the content nodes to download by the types
of content the instance can render. For the content server we want to
fully mirror the channel and download all nodes regardless of whether
the content can be rendered or not.

To do that, the API request is made with the `renderable_only` option
set to `False`. For better or worse, Kolibri 0.15 ignores unknown
options, so this will only do anything useful if the content server
handles the `renderable_only` option[1].

1. learningequality/kolibri#11513

Helps: endlessm/endless-key-content-private#103
By default, Kolibri only imports content nodes that the current instance
has a renderer for. Since we're operating on a homedir that may be
missing plugins that are available at runtime (such as the zim plugin),
it may skip nodes that we ultimately want at runtime. Use the
`--include-unrenderable-content` option so that no nodes are filtered
out.

Helps: endlessm/endless-key-content-private#103
@dylanmccall
Copy link
Contributor

Yep, this makes sense.

@dylanmccall dylanmccall merged commit 3e90d23 into master Nov 9, 2023
2 checks passed
@wjt wjt deleted the kolibri-unrenderable branch November 10, 2023 17:42
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.

2 participants