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

Update landing page info #206

Closed
wants to merge 0 commits into from
Closed

Update landing page info #206

wants to merge 0 commits into from

Conversation

Teddy-1000
Copy link
Collaborator

PR for updating landing page info.

@Teddy-1000 Teddy-1000 requested a review from lukas-phaf October 15, 2024 13:16
Copy link

github-actions bot commented Oct 15, 2024

API Unit Test Coverage Report
FileStmtsMissCoverMissing
\_\_init\_\_.py00100% 
datastore_pb2.py614821%34–81
datastore_pb2_grpc.py542750%15–16, 19, 65–80, 121–123, 128–130, 135–137, 142–144, 148–173, 219, 246, 273, 300
export_metrics.py100100% 
grpc_getter.py201145%15–19, 23–26, 30–32, 36–38
locustfile.py15150%1–31
main.py44784%46, 51, 61, 71–72, 82–83
metadata_endpoints.py683154%49–58, 62, 111, 126–252, 256
response_classes.py50100% 
utilities.py1744674%20, 38, 45, 67–70, 78–89, 94–101, 121, 125, 127, 155, 161, 179, 193–194, 198, 214–218, 222–228, 232–234, 264, 268, 290, 295
custom_geo_json
   edr_feature_collection.py60100% 
formatters
   \_\_init\_\_.py110100% 
   covjson.py59198%94
   geojson.py21290%27, 52
openapi
   custom_dimension_examples.py40100% 
   edr_query_parameter_descriptions.py110100% 
   openapi_examples.py130100% 
   openapi_metadata.py10100% 
routers
   \_\_init\_\_.py00100% 
   edr.py101496%348–349, 438–439
   feature.py471960%99–132, 148–153, 159–181
TOTAL72521171% 

API Unit Test Coverage Summary

Tests Skipped Failures Errors Time
30 0 💤 0 ❌ 0 🔥 1.890s ⏱️

api/openapi/openapi_metadata.py Outdated Show resolved Hide resolved
openapi_metadata = {
"title": "Observations Europe EUMETNET",
"description": (
"EDR data service for European environmental observations from EUMETNET," " co-funded by the European Union"
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't "environmental observations" to broad?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good question. It is the only type of observations we have. I don't really have any better suggestions either.

Copy link
Contributor

Choose a reason for hiding this comment

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

... European meteorological observations ...

api/openapi/openapi_metadata.py Outdated Show resolved Hide resolved
api/openapi/openapi_metadata.py Outdated Show resolved Hide resolved
api/metadata_endpoints.py Outdated Show resolved Hide resolved
api/metadata_endpoints.py Outdated Show resolved Hide resolved
api/metadata_endpoints.py Outdated Show resolved Hide resolved
api/metadata_endpoints.py Outdated Show resolved Hide resolved
"pressure",
"clouds",
"radiation",
],
provider=Provider(name="RODEO", url="https://rodeo-project.eu/"),
contact=Contact(email="[email protected]"),
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want this contact to be different from the one at API level?

@@ -0,0 +1,15 @@
openapi_metadata = {
"title": "Observations Europe EUMETNET",
Copy link
Contributor

Choose a reason for hiding this comment

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

I still don't like this title. It should say "EDR" and "API".

@Teddy-1000 Teddy-1000 closed this Dec 4, 2024
@Teddy-1000 Teddy-1000 force-pushed the update-landingpage-info branch from 628382a to 939cfd0 Compare December 4, 2024 07:09
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.

2 participants