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

updated doc and config #119

Merged
merged 3 commits into from
Dec 7, 2023
Merged

updated doc and config #119

merged 3 commits into from
Dec 7, 2023

Conversation

krishnaglodha
Copy link
Member

Updated bridges.md and pygeoapi.config.yml to include SensorAPI and CSW

Copy link
Contributor

@doublebyte1 doublebyte1 left a comment

Choose a reason for hiding this comment

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

The CSW example is running fine, but there are some issues with the STA bridge. It would be good to test it well, to make sure it works (also from a features client, like QGIS). Feel free to try another STA server, instead of this one.

The bridge section needs a bit more detailed information, in order for the students to be able to run the examples. We are creating documentation, not just code snippets.

@@ -140,8 +140,49 @@ Run the following requests in your web browser:

Copy link
Contributor

Choose a reason for hiding this comment

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

We also need a description of how-to run this. Please look at the features and map examples, in the same section.

workshop/exercises/pygeoapi.config.yml Outdated Show resolved Hide resolved
workshop/exercises/pygeoapi.config.yml Outdated Show resolved Hide resolved
workshop/exercises/pygeoapi.config.yml Outdated Show resolved Hide resolved
workshop/exercises/pygeoapi.config.yml Outdated Show resolved Hide resolved
- type: feature
name: SensorThings
data: https://sensorthings-wq.brgm-rec.fr/FROST-Server/v1.0/
Copy link
Contributor

Choose a reason for hiding this comment

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

Something's not right here. If you browse through the items of this collection, and click on the uri, it will give an exception.

Screenshot 2023-12-06 at 16 11 23

workshop/content/docs/advanced/bridges.md Outdated Show resolved Hide resolved
@doublebyte1 doublebyte1 merged commit 2119f9e into main Dec 7, 2023
1 check passed
@tomkralidis tomkralidis deleted the geopython/issues-109 branch December 11, 2023 14:15
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.

3 participants