gerardojbaez / laraplans

SaaS style recurring plans for Laravel.
http://laraplans.readthedocs.io
MIT License
180 stars 81 forks source link

in laravel 5.5 not working, can you update plz #24

Closed pay2all closed 6 years ago

pay2all commented 6 years ago

in laravel 5.5 not working, can you update plz

oanhnn commented 6 years ago

+1

siarheipashkevich commented 6 years ago

@pay2all, @oanhnn could we please confirm that this problem has been fixed?

gerardojbaez commented 6 years ago

What errors are you guys experiencing? Is it the same reported in #26?

I'm using Laraplans (dev-master) on a project using Laravel 5.5, all tests are passing; maybe the fix is already in place (7f8c23b95788da9a09a0cbe17bea3de5da83a63e), but has not been released yet.

sanjaylive commented 6 years ago
screen shot 2018-01-04 at 4 11 07 pm
gerardojbaez commented 6 years ago

Thanks, @sanjaylive. There's already a commit that solves this error... I will create a release soon. If you are still under development, you can try dev-master to try things out.

siarheipashkevich commented 6 years ago

I've the same issue in laravel 5.5.

sanjaylive commented 6 years ago

check update, already solved

On Fri, Jan 26, 2018 at 8:38 PM, Siarhei Pashkevich < notifications@github.com> wrote:

I've the same issue in laravel 5.5.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/gerardojbaez/laraplans/issues/24#issuecomment-360809592, or mute the thread https://github.com/notifications/unsubscribe-auth/AdTxi0M51yz-Jc53-WGsswUQXh-PaRjXks5tOep0gaJpZM4Q4Ozl .

siarheipashkevich commented 6 years ago

I've switched to dev-master and problem was fixed. But I think we should create a new release.

josteph commented 6 years ago

Please see to this issue (if you're not using dev-master) for the fix. #30

oanhnn commented 6 years ago

You can try my package oanhnn/laravel-pricing-plans

josteph commented 6 years ago

@oanhnn You could have forked this package from @gerardojbaez rather than claiming it as your own.

oanhnn commented 6 years ago

@josteph Yes, it is a fork, but the license of @gerardojbaez 's package is MIT and i did refactor it 😄

gerardojbaez commented 6 years ago

@oanhnn, Having the MIT license doesn't mean you can claim it as yours; it clearly says that you have to preserve the Copyright along with the provided License. Please make the necessary adjustments to your repository. Refactoring someone else code doesn't make it yours.

oanhnn commented 6 years ago

@gerardojbaez I know my project the development on your project and your solution. Thank you very much about it.. But actually I have developed it on my idea and solution from mid-2017. You can now see it is different very much with your project. However, I have linked to your project as the source of the project and added your name to the license in commit afe05cee6fd1c0b1e9f5fbfe672f48dd2cbb4967

gerardojbaez commented 6 years ago

@oanhnn, Thanks for the adjustments. Your project is base on Laraplans, it doesn't matter if you have refactored it or built on top of it, the original Copyright must be included (and you already did). More information here.

This is how I would do it (it is more explicit):

Original work Copyright © 2016 Gerardo Baez <g@gerardobaez.com>
Modified work Copyright © 2017-2018 Oanh Nguyen <oanhnn.bk@gmail.com> 
gerardojbaez commented 6 years ago

The release is out! v2.2.0