-
Notifications
You must be signed in to change notification settings - Fork 48
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
fix(providers): usage of hydrological_year
in provider wekeo_ecmwf
#1313
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is normal that the format of the parameter is different than the other configured for provider wekeo ?
For instance with statistic
:
statistic:
- '{{"statistic": {statistic}}}'
- '$.null'
I used he same format of |
0621732
to
13d3a78
Compare
3031fc8
to
33dec24
Compare
Code Coverage (Ubuntu)
Diff against develop
Results for commit: 9a24db3 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)
Diff against develop
Results for commit: 9a24db3 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
hydrological_year
to provider wekeo_ecmwf
hydrological_year
in provider wekeo_ecmwf
33dec24
to
12c371f
Compare
466edbb
to
9a24db3
Compare
hydrological_year
in provider wekeo_ecmwf
hydrological_year
in provider wekeo_ecmwf
hydrological_year
to providerwekeo_ecmwf
and fix a typo in theorderLink
_dc_qs
for thePostJsonSearch
when downloading the producthydrological_year
Follow examples of queries and the generated
_dc_qs
. These tests can also be found intests.units.test_search_plugins.TestSearchPluginPostJsonSearch.test_plugins_search_postjsonsearch_query_params_wekeo
.Using the
datetime
Generated
_dc_qs
:Using parameter
hydrological_year
(single value)Generated
_dc_qs
:Using parameter
hydrological_year
(multiple values)Generated
_dc_qs
: