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

Enable and disable plugins via the API #4964

Merged
merged 17 commits into from
Jun 5, 2023

Conversation

SchrodingersGat
Copy link
Member

@SchrodingersGat SchrodingersGat commented Jun 3, 2023

This PR does the following:

  • Plugin activation is now handled directly from the settings page
  • Plugin activation uses the API
  • Plugins can now also be deactivated via the API
  • Renders table of plugins via API access, rather than templated on the server side (because who wants templates?)

- Template adjustments
- Allow plugin to be deactivated also
- Default value = True if not provided
- Allow same JS method to either enable or disable a plugin
@SchrodingersGat SchrodingersGat added user interface plugin Plugin ecosystem api Relates to the API labels Jun 3, 2023
@SchrodingersGat SchrodingersGat added this to the 0.12.0 milestone Jun 3, 2023
@SchrodingersGat SchrodingersGat requested a review from matmair as a code owner June 3, 2023 14:45
Copy link
Member

@matmair matmair left a comment

Choose a reason for hiding this comment

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

looks good in general

InvenTree/plugin/serializers.py Outdated Show resolved Hide resolved
@SchrodingersGat SchrodingersGat merged commit 45ec7b9 into inventree:master Jun 5, 2023
@SchrodingersGat SchrodingersGat deleted the plugin-activate branch June 5, 2023 02:20
@SchrodingersGat
Copy link
Member Author

For reference, here's a screenshot of the new interface:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Relates to the API plugin Plugin ecosystem
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants