search

hashids / hashids.github.io

This is the old Hashids website. It is no longer maintained and has migrated over to Sqids.
https://sqids.org
206 stars 34 forks source link

Remove Laravel 4 package #29

Closed vinkla closed 8 years ago

vinkla commented 8 years ago

Laravel 4 is old and the package isn't maintained anymore.

jd327 commented 8 years ago

Laravel 4 might be old, but I'm sure there's still people using it. I don't see a need to remove it.

vinkla commented 8 years ago

People (including myself) are still using it with Laravel 4. Though, new users probably wont use it or shouldn't use it. Anyone using the Laravel 4 integration would still know where to find the package on GitHub. IMO we should encourage new users of Hashids to use the latest version of Laravel. Laravel version 4 and older versions wont see get updates anytime soon.

Also, we're linking to the 4:th version in the Laravel Hashids repository.

jd327 commented 8 years ago

Yeah, all good points, but honestly I don't see the urgency to remove the Laravel 4 link (it's not that old yet). If you want to switch them around so Laravel 5 is first, I'm all good with that. Other than that, it doesn't hurt to still link to it (and new devs are probably well aware that higher version is what they should be using).

vinkla commented 8 years ago

Okay, I see your point. Lets leave it like it is for now.