-
Notifications
You must be signed in to change notification settings - Fork 93
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
Bangladesh OCDID Update #355
Bangladesh OCDID Update #355
Conversation
@azuser Hi Zohour, could you please help review this PR. |
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.
According to this wiki Jatiya Sangsad has 8 divisions. The following districts aren't represented within the additions, please add them:
- Barishal of the Barisal district,
- Jhalokati of the Barisal district
- Moulivibazar of the Sylhet district
identifiers/country-bd/README
Outdated
File contents in `country-bd` directory: | ||
|
||
* Jatiya_Sangsad_electorates.csv:contains the list of electoral districts for the House of the Nation (Jatiya Sangsad) of Bangladesh | ||
|
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.
nit: remove new line from between list elements.
identifiers/country-bd/README
Outdated
@@ -0,0 +1,9 @@ | |||
File contents in `country-bd` directory: | |||
|
|||
* Jatiya_Sangsad_electorates.csv:contains the list of electoral districts for the House of the Nation (Jatiya Sangsad) of Bangladesh |
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.
nit: add a space after the colon.
* Jatiya_Sangsad_electorates.csv:contains the list of electoral districts for the House of the Nation (Jatiya Sangsad) of Bangladesh | |
* Jatiya_Sangsad_electorates.csv: contains the list of electoral districts for the House of the Nation (Jatiya Sangsad) of Bangladesh |
Hi @HKSenior,
|
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.
Thanks for the clarification on the previous divisions.
ocd-division/country:bd/ed:rangamati,Rangamati | ||
ocd-division/country:bd/ed:bandarban,Bandarban | ||
ocd-division/country:bd/ed:khagrachari,Khagrachari | ||
ocd-division/country:bd/ed:coxs_bazar_1,Cox s Bazar-1 |
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.
Hi @HKSenior , we have added apostrophe in Cox's Bazar. |
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.
Did a quick spot check, everything looks good to me.
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.
Looks good. Makes sense to omit the division and district from the OCD-ID, and just keep the constituency. (I quickly compared to https://en.wikipedia.org/wiki/List_of_constituencies_of_the_Jatiya_Sangsad)
Hi @zohour,