Skip to content
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

Update SQL Nigerian > Abuja Cities #562

Closed
wants to merge 3 commits into from
Closed

Update SQL Nigerian > Abuja Cities #562

wants to merge 3 commits into from

Conversation

bensonarafat
Copy link

Updated Abuja cities to the SQL world and cities.
All these were missing at the first DB.
thanks

@w01ki3
Copy link
Contributor

w01ki3 commented Jan 14, 2023

While checking the added lines, I noticed that the wikidataid part is the same in all of them. Why ?

@bensonarafat
Copy link
Author

While checking the added lines, I noticed that the wikidataid part is the same in all of them. Why ?

Oh, my bad..
I just updated it with their vikidataid
thanks for pointing it out.

@bensonarafat
Copy link
Author

bensonarafat commented Jan 17, 2023 via email

@dr5hn
Copy link
Owner

dr5hn commented Feb 12, 2023

@bensonarafat Thanks for your efforts. Really appreciate it.

Can you please rebase the branch, update the cities id and also remove the cities.sql?

dr5hn added a commit that referenced this pull request Mar 6, 2023
…ers & NL - South Holland) States lat/long fixes
@dr5hn dr5hn mentioned this pull request Mar 6, 2023
@dr5hn
Copy link
Owner

dr5hn commented Mar 6, 2023

@bensonarafat Since I've incorporated your changes into PR #578, I will now be closing it. Thank you for your contributions.

@dr5hn dr5hn closed this Mar 6, 2023
dr5hn added a commit that referenced this pull request Mar 6, 2023
@bensonarafat
Copy link
Author

cool
Thanks @dr5hn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants