Skip to content

Migrate according to new hooks architecture in PR#12879#216

Open
ozer550 wants to merge 5 commits intolearningequality:mainfrom
ozer550:migrate-according-to-kolibris-new-hooks-architecture
Open

Migrate according to new hooks architecture in PR#12879#216
ozer550 wants to merge 5 commits intolearningequality:mainfrom
ozer550:migrate-according-to-kolibris-new-hooks-architecture

Conversation

@ozer550
Copy link
Copy Markdown
Member

@ozer550 ozer550 commented Jan 22, 2026

Summary

Updated kolibri-app to work with Kolibri PR #12879, which removed the old app plugin and replaced it with a hook-based system.

  • Removed old plugin dependencies: deleted references to the removed plugin.
  • Updated URL generation: switched to the new function for building initialization URLs.
  • Removed file sharing and metered connection code that wasn’t implemented.

References

closes #212

Reviewer guidance

For Ubuntu:

  • Install system dependencies:
    sudo apt install python3.10-venv python3.10-dev
  • Create and activate a virtual environment:
     cd kolibri-app   
     /usr/bin/python3.10 -m venv venv-system   
     source venv-system/bin/activate
    
  • Install build dependencies:
    make dependencies
  • Download the Kolibri wheel from PR #12879:
    • Go to PR #12879
    • Download the .whl artifact from the CI build
    • Copy the wheel file to the whl/ directory (in kolibri_app):
      mkdir -p whl
      cp /path/to/downloaded/kolibri-*.whl whl/
  • Install the wheel:
    make install-whl whl=whl/kolibri-<version>.whl
  • Run the app:
    make run-dev
  • App should run without any errors.
  • Check logs in KOLIBRI_HOME folder (kolibri.txt, kolibri_app.txts).

Copy link
Copy Markdown
Member

@rtibbles rtibbles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing that needs to be retained.

self.create_hidden_window()

enable_plugin("kolibri.plugins.app")
enable_plugin("kolibri_app")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still need to enable the plugin from this repository! Just not the kolibri one.

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.

Adopt and migrate according to Kolibri’s new app hooks in kolibri-app

2 participants