Skip to content

Conversation

kflemin
Copy link
Contributor

@kflemin kflemin commented Sep 19, 2025

No description provided.

@kflemin kflemin requested review from Copilot and nllong and removed request for nllong September 22, 2025 20:06
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the project to transition from MapQuest to Amazon Location Services for geocoding functionality and refactors the package dependency from cbl-workflow to building-data-utilities.

  • Updates geocoding service to use Amazon Location Services API instead of MapQuest
  • Refactors package imports and dependencies from cbl-workflow to building-data-utilities
  • Enhances footprint merging functionality with improved error handling and JSON serialization

Reviewed Changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
pyproject.toml Updates package dependency and author list
mypy.ini Updates type checking configuration for new package
flask_app/services/geocoding_service.py Changes import from cbl_workflow to building_data_utilities
flask_app/services/footprint_service.py Major refactor of merge functionality with new imports and enhanced processing
flask_app/services/data_transformation_service.py Updates import to new package
flask_app/app.py Switches from MapQuest to Amazon API with enhanced error handling
Documentation files Updates references from MapQuest to Amazon Location Services
Angular components Updates UI text and API service calls
.github/workflows/ci.yml Updates CI to use new package dependency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

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

worked locally!

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