Skip to content

fix: rename EnablePostGIS migration to valid TypeORM filename#90

Merged
snowrugar-beep merged 1 commit into
VertexChainLabs:mainfrom
kilodesodiq-arch:fix/issue-32-enable-postgis-migration
Jun 28, 2026
Merged

fix: rename EnablePostGIS migration to valid TypeORM filename#90
snowrugar-beep merged 1 commit into
VertexChainLabs:mainfrom
kilodesodiq-arch:fix/issue-32-enable-postgis-migration

Conversation

@kilodesodiq-arch

Copy link
Copy Markdown
Contributor

Summary

The EnablePostGIS migration existed in the repository but was saved under an invalid filename (…EnablePostGIS.ts, using a Unicode ellipsis character). TypeORM migration discovery relies on glob pattern matching — this malformed filename caused the migration to be silently skipped, leaving PostGIS uninstalled and all spatial queries failing.

This PR renames the file to the correct TypeORM convention: 1700000000000-EnablePostGIS.ts. The timestamp prefix ensures it runs before CreateGistsTable.ts (timestamp 1700000000001), satisfying the dependency order required by PostGIS spatial functions.

Changes

  • Renamed Backend/src/database/migrations/…EnablePostGIS.tsBackend/src/database/migrations/1700000000000-EnablePostGIS.ts
  • File contents are unchanged — the migration correctly enables both postgis and postgis_topology extensions

Testing

  • Migration file is now discoverable by TypeORM CLI (npm run migration:run)
  • 1700000000000-EnablePostGIS.ts runs before 1700000000001-CreateGistsTable.ts, satisfying the PostGIS dependency
  • Existing tests unaffected

Closes #32

@snowrugar-beep snowrugar-beep left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@snowrugar-beep snowrugar-beep merged commit 629cb5e into VertexChainLabs:main Jun 28, 2026
6 checks passed
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.

infra: create EnablePostGIS database migration file

2 participants