Feature/hdx 9831 hum needs #99
Conversation
…DX-9831-hum-needs
There was a problem hiding this comment.
There is probably a task to review the documentation after we've completed the majority of the migration to the new schema. I think there might be places where we reference endpoints like location which have moved.
There was a problem hiding this comment.
indeed, we will need to review this and probably ask DS team to support us
There was a problem hiding this comment.
Are the age_min and age_max queries deliberately commented out?
There was a problem hiding this comment.
yes, indeed. I am not sure yet if it is going to be one param for each or it will be a min&max for min and min&max for max.
There was a problem hiding this comment.
I think we should be retiring these endpoints since they no longer represent a view - just enums.
There was a problem hiding this comment.
yes, we need a ticket
There was a problem hiding this comment.
I think @alexandru-m-g has dealt with this in #102
There was a problem hiding this comment.
I used the enums in the test fixtures later in this test - where the Response classes are made directly.
danmihaila
merged commit db27e83
into
feature/HDX-9818-prepare-for-new-schema
No description provided.