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

Add support for laravel 11, and add then use the latest php 8.3 version #50

Closed
wants to merge 2 commits into from

Conversation

lloricode
Copy link

additional changes of #49

@lloricode
Copy link
Author

lloricode commented Mar 12, 2024

⚗️ Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.

To do so, temporarily add my fork to the repositories property of your composer.json:

{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/lloricode/json-api.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:

{
    "require": {
        "timacdonald/json-api": "dev-l11-compatibility",
    }
}

Finally, run: composer update

Copy link

@jbrooksuk jbrooksuk left a comment

Choose a reason for hiding this comment

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

@lloricode
Copy link
Author

close as continue at #49

@lloricode lloricode closed this Mar 13, 2024
@lloricode lloricode deleted the l11-compatibility branch March 13, 2024 01:29
@timacdonald
Copy link
Owner

I had to make a few extra tweaks and didn't see this one. Appreciate you sending it through.

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.

3 participants