-
Notifications
You must be signed in to change notification settings - Fork 2
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
HDX-9819 Adapt the dataset and resource endpoints to the new schema #93
HDX-9819 Adapt the dataset and resource endpoints to the new schema #93
Conversation
…dataset_resource_new_schema
'dataset_title': 'Dataset #1', | ||
'dataset_hdx_provider_stub': 'pRoViDeR01', | ||
'dataset_hdx_provider_stub': 'provider01', |
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.
I think this weird looking string, "pRoViDeR01", was there to tests case insensitivity
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.
I think I must have copied it over from elsewhere, I have no problem with the wierd casing.
@@ -301,29 +303,30 @@ | |||
'admin2_name', | |||
'admin1_ref', | |||
], | |||
}, | |||
'/api/resource': { | |||
}, # (hdx_id, dataset_hdx_id, name, format, update_date, is_hxl, download_url, hapi_updated_date) |
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.
is the commented line here on purpose ?
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.
I think it is a remnant of the process I used to make the file, so it can be deleted.
30440db
into
feature/HDX-9818-prepare-for-new-schema
This PR makes updates to the new HAPI schema for the dataset and resource endpoints.
Note that the HAPI API uses title and dataset_title here, as specified in the spreadsheet which is inconsistent with usage elsewhere hdx_name is used (alongside hdx_stub)
I need to add a get_organizational_api_url function and add the related field to the API.