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

feat(geoUtils): add greek cities functions #83

Merged
merged 7 commits into from
Sep 29, 2024

Conversation

arisGio
Copy link
Contributor

@arisGio arisGio commented Aug 31, 2024

Related issue #45 .

getCityAdministrativeDivision using the municipality entity is commented out and needs discussion.

Even though some already existing methods use the "municipality" as default value, only "unit" or "region" level is handled and not the "municipality" one.

Need to elaborate on this case.

Copy link

changeset-bot bot commented Aug 31, 2024

🦋 Changeset detected

Latest commit: 0b1cd44

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@tsevdos/el-utils Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@arisGio
Copy link
Contributor Author

arisGio commented Aug 31, 2024

searchCityByName has not been implemented yet!

@arisGio
Copy link
Contributor Author

arisGio commented Sep 22, 2024

With the latest commit 8efe8de
searchCityByName has been implemented.

@tsevdos tsevdos added the enhancement New feature or request label Sep 26, 2024
@tsevdos tsevdos changed the title enhancement: cities vol 2 feat: cities vol 2 Sep 26, 2024
@tsevdos tsevdos changed the title feat: cities vol 2 feat(geoUtils): add greek cities functions Sep 26, 2024
@tsevdos tsevdos self-requested a review September 26, 2024 06:32
Copy link

sonarcloud bot commented Sep 27, 2024

@tsevdos tsevdos merged commit e08601c into tsevdos:main Sep 29, 2024
6 of 7 checks passed
@github-actions github-actions bot mentioned this pull request Sep 29, 2024
@tsevdos tsevdos mentioned this pull request Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants