Skip to content

Match Kolibri's v2 contentnode and channel filters - #6177

Open
rtibblesbot wants to merge 3 commits into
learningequality:hotfixesfrom
rtibblesbot:issue-6171-28d818
Open

rtibblesbot wants to merge 3 commits into
learningequality:hotfixesfrom
rtibblesbot:issue-6171-28d818

Conversation

@rtibblesbot

@rtibblesbot rtibblesbot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Kolibri browsing Studio got course nodes it excluded; search, question and contains_exercise were ignored; contains_quiz=false still returned quizzes.
  • Public content nodes store modality at publish; a deploy-migrate backfill fills existing ones.
  • v2/contentnode and v2/channel apply Kolibri's filter and search semantics.

References

Fixes #6171.

Reviewer guidance

  • Existing public nodes have NULL modality until deploy-migrate finishes. Until then exclude_modalities=COURSE excludes nothing. Is that acceptable?
  • The backfill is one unbatched UPDATE over every public node with a modality in options. Is that fast enough on production data?
  • keywords now matches each term against title or description, the way Kolibri does. It used to need every term in the title, or every term in the description. The tree endpoint now searches too.

AI usage

Used Claude Code to vendor Kolibri's filters and annotate_modality, and to write the tests and backfill command. Verified with the kolibri_public test suite, flake8, black, makemigrations --check and lintmigrations.

🤖 Generated with Claude Code


@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly

How was this generated?
  • Ran pre-flight CI checks (lint, format, tests) and verified all pass
  • Rebased onto the target branch and resolved any conflicts
  • Reorganized commit history into clean, logical commits
  • Audited the diff to ensure only issue-relevant files are changed
  • Built PR body from the repository's PR template with evidence blocks
@rtibblesbot

🟡 Waiting for feedback

Last updated: 2026-09-25 00:14 UTC

@rtibblesbot
rtibblesbot marked this pull request as ready for review September 24, 2026 23:52
@rtibblesbot
rtibblesbot requested review from a team and marcellamaki September 24, 2026 23:52
rtibblesbot and others added 3 commits September 24, 2026 17:12
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.

1 participant