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

Extract non-Core apps into Hooks-based platform plugins #36184

Open
Tracked by #35144
awais786 opened this issue Jan 28, 2025 · 6 comments
Open
Tracked by #35144

Extract non-Core apps into Hooks-based platform plugins #36184

awais786 opened this issue Jan 28, 2025 · 6 comments

Comments

@awais786
Copy link
Contributor

awais786 commented Jan 28, 2025

  1. ora_staff_grader
  2. program_enrollment
  3. survey
  4. zendesk_proxy
  5. lang_pref
  6. password_policy
  7. geoinfo
  8. ace_common
@awais786
Copy link
Contributor Author

awais786 commented Jan 31, 2025

@kdmccormick Do the above small Django apps (located in openedx/core/djangoapps/) align with the Hook-Based Plugins initiative in Open edX?

Given their size and scope, would they be suitable for conversion into hook-based plugins ?

@jristau1984 jristau1984 moved this from Done to Done - Long Term Storage in Arch-BOM Feb 12, 2025
@awais786
Copy link
Contributor Author

I am working on survey app.

@ormsbee
Copy link
Contributor

ormsbee commented Feb 20, 2025

I'd like to better understand the end goal of this. Is the idea that these would be converted to being hooks-based while staying in edx-platform? Be moved out into a common repo? Separate repos?

@awais786
Copy link
Contributor Author

awais786 commented Feb 25, 2025

The primary objective is to extract them from edx-platform and migrate them to a new repository as hook base plugin. Please let me know if you have any further questions or suggestions.

@kdmccormick
Copy link
Member

@awais786 Sorry for not getting back to you. survey seems like a great app to start with, and your approach of extracting into a separate repo sounds right to me.

@ormsbee
Copy link
Contributor

ormsbee commented Feb 26, 2025

The survey app was built in edx-platform at least partially because it calls into a bunch of things that are not part of any stable APIs or plugin interfaces. I'm worried that we will lift some of these apps out with those functional dependencies in place. If we do that and we're not deprecating them, then we're just making maintenance harder because it will be more difficult to detect regressions and to maintain more repos through upgrade cycles.

There are some things here that could come out easily, like geoinfo and maybe zendesk_proxy (though we should probably see if we can just DEPR Zendesk, because edX built it and I don't think they run it any longer).

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

No branches or pull requests

3 participants