Skip to content

Add TeamServiceProvider with Model Bindings#1155

Merged
curtisdelicata merged 1 commit intomainfrom
sweep/Add-TeamServiceProvider-with-Model-Bindings
May 10, 2025
Merged

Add TeamServiceProvider with Model Bindings#1155
curtisdelicata merged 1 commit intomainfrom
sweep/Add-TeamServiceProvider-with-Model-Bindings

Conversation

@sweep-ai-deprecated
Copy link
Copy Markdown
Contributor

This pull request was created by Sweep to resolve the following request by @curtisdelicata:

@php artisan package:discover --ansi handling the post-autoload-dump event returned with error code 139

Continue chatting at https://app.sweep.dev/c/f4cd7901-5629-499d-b26b-040990b81212.

Purpose

This pull request implements a new TeamServiceProvider to establish proper model bindings between the Laravel GEDCOM package interfaces and our concrete model implementations. This enhancement improves the modularity and maintainability of our genealogy application.

Description

The primary change in this pull request is the implementation of the TeamServiceProvider class that handles dependency injection bindings for various genealogy-related models. The provider maps abstract interfaces from the FamilyTree365/LaravelGedcom package to our concrete model implementations.

The provider includes bindings for several key models including:

  • Batch data processing
  • Family records and events
  • Person records and associations
  • Address information
  • Change tracking
  • Submitter information

This implementation ensures proper dependency resolution and maintains a clean separation between the GEDCOM package interfaces and our application-specific model implementations.

Summary

  • Created new TeamServiceProvider in app/Providers/TeamServiceProvider.php
  • Implemented model bindings for:
    • BatchData
    • Family and related models (FamilyEvent, FamilySlgs)
    • Person and related models (PersonAsso, PersonAlia, PersonEvent)
    • Addr
    • Chan
    • Subm
  • Added proper type declarations and strict typing
  • Implemented register() method with model bindings
  • Added empty boot() method for future extensibility

@curtisdelicata curtisdelicata merged commit 57b4712 into main May 10, 2025
1 of 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.

1 participant