Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework contracts to be compatible with spatie/ignition #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlexVanderbist
Copy link
Contributor

No description provided.

@CodeSkills
Copy link

Will this be merged any time soon? We are using this and would love to upgrade to laravel 9, so if we should wait for this to get merged or to migrate whole code to spatie/laravel-ignition directly?

Thanks for the update!

@stancl
Copy link

stancl commented Feb 13, 2022

Does this PR mean ignition-contracts will always depend on ignition?

I see that there are differences in the namespaces (Spatie\ vs Facade\), but parts of the code, such as class BaseSolution extends \Spatie\Ignition\Contracts\BaseSolution implements Solution, make it seem that way. But I think the dependency should be in the opposite direction?

@PaolaRuby
Copy link
Contributor

@stancl yes, but they are trying to leave development of this package behind so ignition will be the main fork, this PR is for compatibility with older apps

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.

4 participants