search

cncf / cncf.io

☁️♮🏛🚧 The CNCF.io WordPress website
https://cncf.io
MIT License
84 stars 38 forks source link

People import fails for countries with commas #888

Open cjyabraham opened 1 month ago

cjyabraham commented 1 month ago

We split the "location" field in the people.json file on the commas to find the country as the last element of this array. The problem is that some of the countries in our database have commas in them, such as "Moldova, Republic of", so there's no way for anyone to find a match with this country. We have a person who has put "location": "Chisinau, Moldova" and another who has "location": "Chisinau, Republic of Moldova" so we need to find a way to work with these entries. There are other countries too that will run into this problem.

cjyabraham commented 1 month ago

For now, I've edited "Moldova, Republic of" to be just "Moldova" in both the CNCF site and the people json file so now they are matching correctly. I'll leave this open but demote priority in case we need a more general solution to accommodate commas later.