-
Notifications
You must be signed in to change notification settings - Fork 143
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
Added RO cities list #168
base: master
Are you sure you want to change the base?
Added RO cities list #168
Conversation
This list is updated as of January 2021.
Hello @rwkyyy, Before adding this pull request to |
It's part of the language, that's how they are spelled. |
So, if I understand correctly, it's okay to replace the current names, with special characters like Bălești-Cătun, with the ones you proposed, like Balesti-Catun, right? I can't confirm this with anyone else in Romania, so I'm trusting what you tell me. |
No, you can not. It's considered lack of seriousness when not using them in a website. |
In such case I need to ask you to update the names with the correct characters, so I can proceed to merge your pull request. |
The reason why I submitted them in this format, is because I did not know if you have a filter for removing them; some shipping service providers do not recognize them correctly and this causes a issue for the end-use.r |
Great update. |
@Killeros sure |
https://easyupload.io/88kgt2 Multumesc de ajutor. |
@YordanSoares can you please add this or you need to patch the language diacritics? |
Hi @rwkyyy, We are planning to resume the places merging very soon. Is this PR still valid to merge? |
@YordanSoares from my knowledge yes, it should still be valid |
This list is updated as of January 2021.