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

import Admin module based on runtime configuration (#1469) #1485

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

tomkralidis
Copy link
Member

Overview

Loads Admin module conditionally.

Related issue / discussion

Fixes #1469

Additional information

None

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@tomkralidis tomkralidis changed the title import Admin module based on runtim configuration (#1469) import Admin module based on runtime configuration (#1469) Jan 9, 2024
@tomkralidis tomkralidis force-pushed the issue-1469 branch 2 times, most recently from 83297fd to edfa944 Compare January 9, 2024 21:21
Copy link
Member

@webb-ben webb-ben left a comment

Choose a reason for hiding this comment

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

+1

@tomkralidis
Copy link
Member Author

CI failure was intermittent. Merging.

@tomkralidis tomkralidis merged commit 635562f into master Jan 9, 2024
10 of 12 checks passed
@tomkralidis tomkralidis deleted the issue-1469 branch January 9, 2024 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ModuleNotFoundError: No module named 'jsonpatch'
2 participants