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

RequestHandlerDelegate.contentTypes and RequestHandler.links #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rldhont
Copy link
Contributor

@rldhont rldhont commented Sep 3, 2021

To provide self and alternate links the RequestHandlerDelegate.contentTypes and RequestHandler.links have been implemented because QgsServerOgcApiHandler::contentTypes and QgsServerOgcApiHandler::links are not available in Python bindings.

This PR is an extraction of #7

@rldhont rldhont added the enhancement New feature or request label Sep 3, 2021
@rldhont rldhont requested a review from dmarteau September 3, 2021 13:00
@dmarteau
Copy link
Member

dmarteau commented Sep 3, 2021

Some of the proposed fixes and changes are actually at hand. This PR is not relevant anymore.

@dmarteau dmarteau added the duplicate This issue or pull request already exists label Sep 3, 2021
… types this handler can serve, default to JSON.

QgsServerOgcApiHandler::contentTypes not available in Python bindings
…given request

The QgsServerOgcApiHandler::links not available in Python bindings
@rldhont rldhont force-pushed the request-handler-links branch from c86f150 to 4f466fc Compare September 3, 2021 15:32
@rldhont
Copy link
Contributor Author

rldhont commented Sep 3, 2021

@dmarteau which code are duplicated ?

Copy link
Member

@dmarteau dmarteau left a comment

Choose a reason for hiding this comment

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

@dmarteau which code are duplicated ?

Layers path fix

return self._parent.href(self._context,path,extension)
return self._parent.href(self._context, path, extension)

def links(self) -> List[str]:
Copy link
Member

Choose a reason for hiding this comment

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

There is no point to implement this here, apiutils is not a global library and will never be one because of the way Qgis plugin are distributed (with no dependencies)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants