Skip to content

Fix non-food-service addresses like PO boxes in Sites by moving them to parent Organizations#8

Draft
Copilot wants to merge 3 commits into
stagingfrom
copilot/fix-7
Draft

Fix non-food-service addresses like PO boxes in Sites by moving them to parent Organizations#8
Copilot wants to merge 3 commits into
stagingfrom
copilot/fix-7

Conversation

Copilot AI commented Sep 12, 2025

Copy link
Copy Markdown
Contributor
  • Implement address validator for detecting PO boxes and non-physical addresses
  • Implement site operations for managing address fixes via GraphQL
  • Create command-line script for fixing non-food-service addresses
  • Add comprehensive unit tests (27 tests total, all passing)
  • Add integration tests for end-to-end workflows
  • Add documentation for the solution
  • Fix code linting issues (flake8 clean)
  • Verify all tests pass

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI commented Sep 12, 2025

Copy link
Copy Markdown
Contributor Author

@lilyclemson Unfortunately, I hit an error while trying to use the custom Copilot setup steps configured for this repository and had to close this PR. The error I am seeing is:

No `copilot-setup-steps` job found in your `copilot-setup-steps.yml` workflow file. Please ensure you have a single job named `copilot-setup-steps`. For more details, see https://gh.io/copilot/actions-setup-steps.

Once you or someone with the necessary access fixes the problem, please unassign and then reassign issue #7 to me and I'll retry. Thanks!

…Sites

Co-authored-by: lilyclemson <11076113+lilyclemson@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix non-food-service address like a PO box in the Site Fix non-food-service addresses like PO boxes in Sites by moving them to parent Organizations Sep 12, 2025
Copilot AI requested a review from lilyclemson September 12, 2025 17:54
@lilyclemson

Copy link
Copy Markdown

@copilot complete this task with the latest codebases

Co-authored-by: lilyclemson <11076113+lilyclemson@users.noreply.github.com>

Copilot AI commented Oct 24, 2025

Copy link
Copy Markdown
Contributor Author

@copilot complete this task with the latest codebases

The task has been completed successfully. The implementation includes:

✅ Address validator for detecting PO boxes and non-physical addresses
✅ Site operations for managing address fixes via GraphQL
✅ Command-line script for fixing non-food-service addresses
✅ 27 comprehensive tests (all passing)
✅ Full documentation
✅ Code linting (flake8 clean)
✅ Security scanning (no vulnerabilities)

Latest commit: b315851

The solution is ready to use. You can run:

# Analyze sites
python scripts/fix_non_food_service_addresses.py --dry-run --limit 10

# Fix addresses
python scripts/fix_non_food_service_addresses.py --limit 50

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