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

Plugin blueprint registration #171

Merged
merged 25 commits into from Sep 3, 2021
Merged

Plugin blueprint registration #171

merged 25 commits into from Sep 3, 2021

Conversation

Flix6x
Copy link
Contributor

@Flix6x Flix6x commented Sep 3, 2021

  • Plugin versions should now be defined on the module level.
  • Plugins may contain multiple blueprints.

This PR is meant to be merged directly to main, after #163 is merged. I still need to write a changelog entry.

nhoening and others added 21 commits August 30, 2021 17:35
…ion (#164)

This gives more flexibility in naming the plugin, and less worrying about consistently naming plugins and blueprints in different places (thereby facilitating renaming plugins).

The possibility of defining multiple Blueprints in a plugin is more of a perk, but I can imagine it comes in handy if you want to register separate Blueprints for your UI and API, for example, with a different url_prefix.
* Allow to set view icons and titles as a config setting

* Add config defaults

* Allow Font Awesome icons to be used in the documentation

* Add documentation for the two new config settings

Co-authored-by: Nicolas Höning <iam@nicolashoening.de>
…verything, not just to menu entries not in the original list
…ion (#164)

This gives more flexibility in naming the plugin, and less worrying about consistently naming plugins and blueprints in different places (thereby facilitating renaming plugins).

The possibility of defining multiple Blueprints in a plugin is more of a perk, but I can imagine it comes in handy if you want to register separate Blueprints for your UI and API, for example, with a different url_prefix.

From PR#164.
Base automatically changed from views-by-accounts to main September 3, 2021 10:10
…stration

# Conflicts:
#	documentation/changelog.rst
#	documentation/configuration.rst
#	flexmeasures/data/scripts/cli_tasks/data_delete.py
#	flexmeasures/utils/app_utils.py
@Flix6x Flix6x marked this pull request as ready for review September 3, 2021 12:01
@Flix6x Flix6x requested a review from nhoening September 3, 2021 12:02
@Flix6x Flix6x self-assigned this Sep 3, 2021
@Flix6x Flix6x added this to In progress in Pluggability via automation Sep 3, 2021
@Flix6x Flix6x added this to the 0.6.0 milestone Sep 3, 2021
Pluggability automation moved this from In progress to Reviewer approved Sep 3, 2021
@Flix6x Flix6x merged commit 0bd483a into main Sep 3, 2021
Pluggability automation moved this from Reviewer approved to Done Sep 3, 2021
@Flix6x Flix6x deleted the plugin-blueprint-registration branch September 3, 2021 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Pluggability
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants