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

Register multiple blueprints for each plugin #4495

Merged
merged 3 commits into from Oct 16, 2018

Conversation

Projects
None yet
3 participants
@alycejenni
Copy link
Contributor

alycejenni commented Oct 11, 2018

Proposed fixes:

Allows returning a list of Blueprint objects from the IBlueprint.get_blueprint() method in order to register multiple separate blueprints per plugin; useful for plugins with a lot of new routes.

Features:

  • includes tests covering changes
  • includes updated documentation
  • includes user-visible changes
  • includes API changes
  • includes bugfix for possible backport

Please [X] all the boxes above that apply

alycejenni added some commits Jul 30, 2018

[sync]
update to latest
Allow plugins to define multiple blueprints
If `IBlueprint.get_blueprint()` returns a list, register the `Blueprint` objects in a loop. Backwards compatible; will still work if the method only returns a single `Blueprint`.

@amercader amercader self-assigned this Oct 16, 2018

@amercader amercader merged commit 147776b into ckan:master Oct 16, 2018

2 checks passed

ci/circleci: test Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@amercader

This comment has been minimized.

Copy link
Member

amercader commented Oct 16, 2018

Thanks @alycejenni this is really good. We'll backport to the current version.

@alycejenni alycejenni deleted the alycejenni:register-multiple-blueprints branch Oct 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment