kolibri
kolibri copied to clipboard
learningequality
Use new content management utility functions for import channel and delete content
While the recent refactoring of content management commands (see #9266) has improved our codebase, there are still instances where the legacy approach is used. Specifically, in the following locations: DeleteContent Delete Content ImportChannel
Instead of invoking the management commands in these cases, we should update the code to use the corresponding utility functions:
For importing a channel, use:
content.utils.channel_transfer.transfer_channel
For deleting content, use:
content.utils.content_delete.delete_content
This change will keep our code up-to-date with our latest updates and make unit testing must simpler
Hi @manuaishika - I have assigned you as you just got in first, please let me know how you plan to tackle this issue.
@KumarVivekPathak are you serious? https://github.com/learningequality/kolibri/blob/develop/kolibri/core/content/utils/content_delete.py
It's right there.
This makes me suspect that this github comment was generated by an LLM - please consider the content of what you post and don't use the output of an LLM without any critical reflection.
Automatically unassigning @manuaishika due to no comments here, or updates on the associated pull request for 1 month. @manuaishika, if you're still interested in this issue or already have work in progress, please message us here, and we'll assign you again. Thank you!
![learning-equality-bot[bot] avatar](https://avatars.githubusercontent.com/in/965472?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hi @rtibbles @thesujai, can I help with this? Also just to be sure, I'm assuming there are other instances with legacy code apart from the ones mentioned in the issue?
Hi @iamshobhraj! I will assign this issue to you! For now, let's stick to the scope of the issue and resolve only the places the issue mentions, regardless of whether there are other instances with legacy code.
