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

Implement DJP: Django Plugins. #4201

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Implement DJP: Django Plugins. #4201

wants to merge 1 commit into from

Conversation

frjo
Copy link
Contributor

@frjo frjo commented Nov 8, 2024

Read up on DJP at https://djp.readthedocs.io/

@frjo frjo added Type: Feature This is something new (not an enhancement of an existing thing). Type: Minor Minor change, used in release drafter labels Nov 8, 2024
@frjo frjo requested a review from theskumar November 8, 2024 11:26
@theskumar
Copy link
Member

Looks good.

@theskumar
Copy link
Member

You can also add folder based discovery, so if someone drop a plugin a specific folder it will be automatically picked up.

Benefit I see you don't do installed them.

@theskumar
Copy link
Member

You can also add folder based discovery, so if someone drop a plugin a specific folder it will be automatically picked up.

Benefit I see you don't do installed them.

I thought it needed any configuration in host application, but looks like nothing needs to be done.

https://djp.readthedocs.io/en/latest/installing_plugins.html#loading-plugins-from-a-directory

@frjo frjo marked this pull request as ready for review November 22, 2024 08:34
@frjo frjo added Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team Status: Needs testing Tickets that need testing/qa Type: Feature This is something new (not an enhancement of an existing thing). Type: Minor Minor change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants