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

Added Laravel 11 Supports #239

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added Laravel 11 Supports #239

wants to merge 2 commits into from

Conversation

akr4m
Copy link

@akr4m akr4m commented Mar 12, 2024

Laravel 11 is recently released.
This PR adds support for Laravel 11.

image

@alies-dev
Copy link
Contributor

alies-dev commented Mar 14, 2024

for anyone looking for a maintained alternative, there is a fork that supports L11, fully compatible with this package and has some features on the top: https://github.com/InteractionDesignFoundation/laravel-geoip

a super simple upgrade guide: https://github.com/InteractionDesignFoundation/laravel-geoip/blob/main/docs/migration.md

@paceband
Copy link

paceband commented Mar 14, 2024 via email

@adnane-ka
Copy link

Is there any conflicting code or it's just okey to go with composer.json dependencies update?

@alies-dev
Copy link
Contributor

alies-dev commented Jul 12, 2024

@adnane-ka

Is there any conflicting code or it's just okey to go with composer.json dependencies update?

If you are talking about https://github.com/InteractionDesignFoundation/laravel-geoip, then - it's a fork without breaking changes. Just update package name in composer.json:

-"torann/geoip": "^3.0",
+"interaction-design-foundation/laravel-geoip": "^3.0",

The only breaking change is a new namespace:

-Torann\GeoIP
+InteractionDesignFoundation\GeoIP

Plus there are some new features and about 10 fixes, you can read more here: https://github.com/InteractionDesignFoundation/laravel-geoip/releases

It's documented in the upgrade guide: https://github.com/InteractionDesignFoundation/laravel-geoip/blob/main/docs/migration.md

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