Add english language and post office support#98
Merged
Conversation
Refs: RATYK-154
Update list display, ordering and search fields for: - MunicipalityAdmin - StreetAdmin - PostalCodeAreaAdmin - AddressAdmin Refs: RATYK-154
Update importers to use english language for: - Street - Municipality - Postal code area Fallback-logic explained: - sv: use finnish language if swedish translation is empty - fi: use swedish language if finnish translation is empty - en: use finnish or swedish version as english Refs: RATYK-154
Updates: - Use explicit source srid constants to make code more readable. - Rely on address data update only by default. - Rename "postal code import" to "postal code area import". Refs: RATYK-163
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
mhieta
force-pushed
the
RATYK-154/update-translations
branch
from
0993fb5 to
79fb45a
Compare
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
Add a new Django management command to delete all address data from the database (municipalities, streets, addresses and postal code areas). Refs: RATYK-163
- Add shell script wrapper for delete_address_data management command - Include safety confirmation prompt before deletion - Follow existing script conventions and shellcheck compliance - Make script executable with proper permissions Refs: RATYK-163
Digiroad-import: - Update URL - Allow curl to follow redirects - Fix multiple shapefile argument support Paavo-import: - Use WFS 2.0 - Use typeName 'pno_meri_2026' - Update management command name Update documentation from the underlying management commands. Refs: RATYK-163
BREAKING CHANGE: municipality-field is no longer accessible in PostalCodeArea after this change. Details: - Update PostalCodeArea-model - Add migration - Update API - Update imports - Update tests - Update Django Admin Refs: RATYK-159
Refs: RATYK-157
Replace distinct() with subqueries for performance. Also update OpenApiParameters. Refs: RATYK-160
mhieta
force-pushed
the
RATYK-154/update-translations
branch
from
79fb45a to
6bf0715
Compare
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
Details: - Add post_office field with fi/sv/en translations to PostalCodeArea model - Implement import command for Posti ZIP/DAT format with fallback logic - Add postoffice query parameter to Address and PostalCodeArea APIs - Update finnish translations - Add comprehensive test coverage Refs: RATYK-157
- Create convenience script for municipality data import - Add file existence validation and clear error messages - Include usage instructions with NLS download link - Require shapefile path as command-line argument - Set executable permissions and verify with shellcheck Refs: RATYK-160
Refs: RATYK-160
Exclude migrations and pipelines from the Sonar analysis. Refs: RATYK-160
mhieta
force-pushed
the
RATYK-154/update-translations
branch
from
6bf0715 to
c013bd9
Compare
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
Province is only needed in the shell-script, therefore remove it from the management command. Refs: RATYK-163
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
danipran
approved these changes
Feb 20, 2026
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
Refs: RATYK-163
Use dictionary comparisons instead of individual field assertion for cleaner and more maintainable code. Refs: RATYK-163
Validate that municipality code exists in the province's municipality list. Refs: RATYK-163
mhieta
force-pushed
the
RATYK-154/update-translations
branch
from
4a0f478 to
06c16d9
Compare
|
|
PAIKKATIETOHAKU-API branch is deployed to platta: https://paikkatietohaku-api-pr98.api.dev.hel.ninja 🚀🚀🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Municipality,StreetandPostalCodeAreapost_officefield toPostalCodeAreawith API filteringimport_post_officesDjango management command and a wrapper shell-script for Posti ZIP/DAT formatdelete_address_dataDjango management command and a wrapper shell-script to delete all address data from the database (municipalities, streets, addresses and postal code areas).BREAKING CHANGE:
municipality-field is removed fromPostalCodeAreabecause the underlying Tilastokeskus-dataset contains only one municipality per PostalCodeArea and there are special cases, where multiple municipalities are needed per one PostalCodeArea.Refs: RATYK-160