PR HDX-9844 national risk endpoint #101
Conversation
There was a problem hiding this comment.
I guess we see location here because this dataset does not use admin1 and admin2
There was a problem hiding this comment.
I need to make sure the generated views for hdx-class use the enums we see here (RiskClass)
There was a problem hiding this comment.
I'm wondering whether we should remove the un-versioned roots - they don't represent compatibility with the legacy version. The only reason to keep them is for a route to the "latest" version.
There was a problem hiding this comment.
I must admit I don't like this pattern of having a "srv" function which does nothing but call the "view_list" function with all its arguments. I guess it gives us a place to change / modify arguments between user and query logic.
There was a problem hiding this comment.
This is just a bit of tidying up
…DX-9844-national-risk
danmihaila
merged commit 53c6766
into
feature/HDX-9818-prepare-for-new-schema
national risk endpoint