opencdms-dev / legacy-opencdms-api

⭐🐍 OpenCDMS server application written in Python (FastAPI) and exposing a web interface for `opencdms-app` and other applications
MIT License
3 stars 3 forks source link

Climsoft station api #11

Closed faysal-ishtiaq closed 2 years ago

faysal-ishtiaq commented 2 years ago

Merge conflicts are resolved. But the tests are not working yet.

isedwards commented 2 years ago

@faysal-ishtiaq could you give a quick updates on which Climsoft tables are now supported and which ones remain?

We can either prioritse the order for implementing the final tables --- or, if all of the main tables that we need are already supported then we may move on to a couple of other issues first.

faysal-ishtiaq commented 2 years ago

@isedwards

The following climsoft tables have CRUD:

There are only a few remaining tables. They will be done by tonight.

faysal-ishtiaq commented 2 years ago

@isedwards

just finished till integration tests for all models. starting snapshot tests

faysal-ishtiaq commented 2 years ago

@isedwards

just gone through the target tables from test db in opencdms-test-data repository. Now adding more information on read me.

faysal-ishtiaq commented 2 years ago

@isedwards

I have just fixed mch db issue. Here is a demo request and response using curl:

04:34:40 faysal@pop-os ~ → curl -X 'GET' \
>   'http://localhost:8000/mch/API/stations' \
>   -H 'accept: application/json' \
>   -H 'Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiJzdHJpbmciLCJleHAiOjE2Mzg2MTQwMzYsInRva2VuX3R5cGUiOiJhY2Nlc3MiLCJqdGkiOiJhZmM5YzU0MzI5NDY0MDA5OWM0ZGVlM2IzNzk1OTFhZSIsInVzZXJfaWQiOjF9.PDk23tWTzUTK-FOsNgcVhZcG9vybp1te1JGsoyg2zRc'

response:

[]