Skip to content

Fix postgis_db paths after move and add Socrata sync probe - #762

Merged
gregpawin merged 1 commit into
hackforla:mainfrom
gregpawin:feat/postgis-paths-and-socrata-sync
Sep 22, 2026
Merged

gregpawin merged 1 commit into
hackforla:mainfrom
gregpawin:feat/postgis-paths-and-socrata-sync

Conversation

@gregpawin

Copy link
Copy Markdown
Member

Summary

  • Update postgis_db docs and script comments for the projects/data-science/ layout after Move data-science/ to projects/data-science/ #759.
  • Fix postgis_db/dumps/* gitignore so probe/dump artifacts are ignored while keeping .gitkeep.
  • Add a dry-run/apply Socrata → PostGIS sync probe (scripts/test_socrata_sync.py) ported from the beta_pipeline SQLite sync idea.
  • Repair broken Jinja conditionals in the citation explorer template (index.html).

Test plan

  • Confirm README / VPS paths match projects/data-science/postgis_db
  • Confirm git check-ignore matches files under postgis_db/dumps/ (except .gitkeep)
  • Dry-run: scripts/test_socrata_sync.cmd (or .py) against local PostGIS with SOCRATA_APP_TOKEN
  • Load explorer UI and verify region select / single vs compare mode still work
  • Optional: run pytest tests/test_service_integration.py -m integration with PostGIS up

Made with Cursor

…ucture

- Updated paths in `README.md` to reflect the move from `data-science/` to `projects/data-science/`.
- Adjusted commands in `VPS.md` for consistency with the new directory structure.
- Enhanced integration test instructions in `test_service_integration.py` to align with the updated paths.
- Modified `app.py` in the web sheet to reference the new project structure.
- Updated `smoke_test.cmd` permissions to ensure executable status.

This commit ensures all documentation and scripts are consistent with the recent refactor, improving clarity for users and developers.
@github-actions

Copy link
Copy Markdown

@gregpawin, this Pull Request is not linked to a valid issue. Please provide a valid linked issue in "Related Issues" above, using the format of "Resolves #" + issue number.

@gregpawin
gregpawin merged commit 5a9ecc6 into hackforla:main Sep 22, 2026
2 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.

2 participants