personal-projects/personal-portfolio
Template
1
1
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Delete legacy TRREB Python modules (schemas, parsers, loaders) #47

New Issue
Closed
opened 2026-01-16 14:54:06 +00:00 by lmiranda · 1 comment
lmiranda commented 2026-01-16 14:54:06 +00:00
Owner
Copy Link

Description

Remove all TRREB-specific Python modules that are no longer needed after transitioning to the neighbourhood-centric dashboard architecture.

Branch: feature/9-cleanup

Reference: docs/changes/Change-Toronto-Analysis-Reviewed.md - Phase 1

Files to DELETE

File Reason
portfolio_app/toronto/schemas/trreb.py TRREB schema obsolete
portfolio_app/toronto/parsers/trreb.py PDF parsing no longer needed
portfolio_app/toronto/loaders/trreb.py TRREB loading logic obsolete

Acceptance Criteria

  • portfolio_app/toronto/schemas/trreb.py deleted
  • portfolio_app/toronto/parsers/trreb.py deleted
  • portfolio_app/toronto/loaders/trreb.py deleted
  • No import errors when running python -c "from portfolio_app.toronto import schemas, parsers, loaders"
  • make test passes (after test cleanup in separate issue)

Technical Notes

  • These files contain TRREB PDF parsing logic that is replaced by Toronto Open Data API integration
  • Ensure no other modules import from these files before deletion
  • Related to but separate from: module __init__.py updates

Sprint

Sprint 9 - Phase 1 (Cleanup)

Labels: Type/Refactor, Priority/High, Efforts/S, Component/Backend, Tech/Python

## Description Remove all TRREB-specific Python modules that are no longer needed after transitioning to the neighbourhood-centric dashboard architecture. **Branch:** `feature/9-cleanup` **Reference:** `docs/changes/Change-Toronto-Analysis-Reviewed.md` - Phase 1 ## Files to DELETE | File | Reason | |------|--------| | `portfolio_app/toronto/schemas/trreb.py` | TRREB schema obsolete | | `portfolio_app/toronto/parsers/trreb.py` | PDF parsing no longer needed | | `portfolio_app/toronto/loaders/trreb.py` | TRREB loading logic obsolete | ## Acceptance Criteria - [ ] `portfolio_app/toronto/schemas/trreb.py` deleted - [ ] `portfolio_app/toronto/parsers/trreb.py` deleted - [ ] `portfolio_app/toronto/loaders/trreb.py` deleted - [ ] No import errors when running `python -c "from portfolio_app.toronto import schemas, parsers, loaders"` - [ ] `make test` passes (after test cleanup in separate issue) ## Technical Notes - These files contain TRREB PDF parsing logic that is replaced by Toronto Open Data API integration - Ensure no other modules import from these files before deletion - Related to but separate from: module `__init__.py` updates ## Sprint Sprint 9 - Phase 1 (Cleanup) **Labels:** Type/Refactor, Priority/High, Efforts/S, Component/Backend, Tech/Python
lmiranda added this to the Launch: Host, Bio and Toronto House Market Analysis project 2026-01-16 14:55:49 +00:00
lmiranda self-assigned this 2026-01-16 14:55:54 +00:00
lmiranda moved this to To Do in Launch: Host, Bio and Toronto House Market Analysis on 2026-01-16 14:55:59 +00:00
lmiranda commented 2026-01-16 15:01:13 +00:00
Author
Owner
Copy Link

✅ Completed in commit cb877df

Deleted:

  • portfolio_app/toronto/schemas/trreb.py
  • portfolio_app/toronto/parsers/trreb.py
  • portfolio_app/toronto/loaders/trreb.py

Updated __init__.py files to remove TRREB imports. Package imports verified working.

✅ **Completed** in commit `cb877df` Deleted: - `portfolio_app/toronto/schemas/trreb.py` - `portfolio_app/toronto/parsers/trreb.py` - `portfolio_app/toronto/loaders/trreb.py` Updated `__init__.py` files to remove TRREB imports. Package imports verified working.
lmiranda moved this to Done in Launch: Host, Bio and Toronto House Market Analysis on 2026-01-16 15:33:49 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Complexity/Complex
High uncertainty or many dependencies
 Complexity/Medium
Some unknowns or dependencies
 Complexity/Simple
Straightforward, well-understood
 Effort/L
3-5 days
 Effort/M
1-2 days
 Effort/S
2-4 hours
 Effort/XL
> 1 week
 Effort/XS
< 2 hours
 Priority/Critical
Immediate attention required
 Priority/High
Important, needs attention soon
 Priority/Low
Nice to have, can wait
 Priority/Medium
Normal priority
 Type/Bug
Something isn't working
 Type/Chore
Maintenance tasks
 Type/Documentation
Documentation improvements
 Type/Feature
New feature or request
 Type/Refactor
Code refactoring without changing behavior
 Type/Test
Testing related
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project Launch: Host, Bio and Toronto House Market Analysis
Assignees
Clear assignees
lmiranda
No Assignees lmiranda
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: personal-projects/personal-portfolio#47
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?