map-egypt / map-egypt.github.io

Other
1 stars 6 forks source link

Indicators aren't showing up in Arabic #345

Closed felskia closed 6 years ago

felskia commented 7 years ago

Right now it says 0 but there are a bunch.

screen shot 2017-08-14 at 4 14 47 pm
dereklieu commented 7 years ago

The API is not returning categories in arabic, @kamicut can you look to see why? Without Arabic categories there's no way to group the indicators. It also seems that the indicator names are coming in only English.

kamicut commented 7 years ago

@dereklieu I'm correcting the arabic names in the API now. By categories do you mean themes? There are two types of "categories", the "type" of the indicator (sequential, diverging, categorical), and the "theme": "Economy", "Agriculture", which has the type "other", but also SDS and SDG indicators.

kamicut commented 7 years ago

@felskia @dereklieu The labels.js label file that is in the admin does not contain the proper translation for themes. There are 2 missing translations for the SDS indicators, and no translations for SDG and "other" themes. We need these translations to render them in the frontend.

kamicut commented 6 years ago

@dereklieu I've added the missing translations to the labels file. Could you check if this is enough to continue work on this ticket?

dereklieu commented 6 years ago

@kamicut Not yet, the name_ar and description_ar fields in this response are still null:

https://ifpri-egypt-silo.herokuapp.com/indicators

kamicut commented 6 years ago

@dereklieu that's a data entry mistake by the data team. There are some indicators (for example "Rice Production - 2010") that do have entries. Could we filter out the nulls?

dereklieu commented 6 years ago

It is filtering out the nulls, the problem is most of them are null, which is how we get the display at the top of this ticket.

@felskia it looks like this is data-entry related, the arabic names aren't getting inputted into the api somehow.

felskia commented 6 years ago

So this would be a @kamicut problem? We keep getting pinged by them since these little changes have taken us a while. Is there anyway the two of you can wrap these things up this week or beginning of next?

@dereklieu

dereklieu commented 6 years ago

Closed #352