Skip to content

Implement DJP: Django Plugins. - #4201

Merged
frjo merged 1 commit into
mainfrom
feature/add-djp
Nov 25, 2024
Merged

Implement DJP: Django Plugins.#4201
frjo merged 1 commit into
mainfrom
feature/add-djp

Conversation

@frjo

@frjo frjo commented Nov 8, 2024

Copy link
Copy Markdown
Member

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
Copy Markdown
Member

Looks good.

@theskumar

Copy link
Copy Markdown
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
Copy Markdown
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
@wes-otf

wes-otf commented Nov 25, 2024

Copy link
Copy Markdown
Contributor

Going to do some studying up on this in the next coming weeks but this looks good!

@wes-otf wes-otf added Status: Tested - approved for live ✅ and removed Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Nov 25, 2024
@frjo
frjo merged commit dec8d27 into main Nov 25, 2024
@theskumar
theskumar deleted the feature/add-djp branch March 11, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Tested - approved for live ✅ 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.

3 participants