Skip to content

vocab + psql CRUD updates #1

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

vocab + psql CRUD updates #1

wants to merge 5 commits into from

Conversation

david-i-berry
Copy link
Collaborator

Overview

Addition of vocab route and non spatial postgresql provider.
HTML template for vocab pages
Addition of license information
CRUD operations for psql

Related Issue / Discussion

Additional Information

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

Addition of /vocab route to flask app. New postgresql non spatial provider.

Update to plugin name

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

Update to collection to add in license information if present.

CRUD operations for psql.

CRUD operations for psql.

CRUD operations for psql.

CRUD operations for psql.

shapely 2

shapely 2

debugging

debugging

debugging

debugging

debugging

debugging

debugging

sqlalchemy < 2 !

testing

testing

addition of delete

addition of delete
@david-i-berry david-i-berry changed the title Update .gitignore vocab + psql CRUD updates Apr 26, 2023
@david-i-berry david-i-berry requested a review from isedwards April 26, 2023 11:53
@cla-bot cla-bot bot added the cla-signed label Apr 27, 2023
@david-i-berry
Copy link
Collaborator Author

@isedwards

Choose a reason for hiding this comment

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

@david-i-berry - can you remind me why we need the autoescape fix and whether this is something that could be contributed directly to pygeoapi instead?

Copy link

@isedwards isedwards left a comment

Choose a reason for hiding this comment

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

With the exception of the autoescape fix in util.py mentioned below, the changes made in opencdms-pygeoapi have now been migrated into the main opencdms repo.

Instead of being a fork of pygeoapi, the new version (in opencdms/api) reimplements pygeoapi serve and flask_app.py - and then, for everything else, we either import directly from pygeoapi or, where required, we import our own version.

pygeoapi serve is reimplemented as opencdms api serve.

This means that we no longer have to maintain a complete fork of the original software. Also the opencdms-pygeoapi and opencdms-tmp repos are no longer required (opencdms-tmp is now in opencdms/install).

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

Successfully merging this pull request may close these issues.

2 participants