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

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

Closed
pay2all opened this issue Dec 6, 2017 · 16 comments
Closed

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

pay2all opened this issue Dec 6, 2017 · 16 comments
Labels

Comments

@pay2all
Copy link

pay2all commented Dec 6, 2017

in laravel 5.5 not working, can you update plz

@oanhnn
Copy link

oanhnn commented Dec 14, 2017

+1

@siarheipashkevich
Copy link

siarheipashkevich commented Jan 2, 2018

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

@gerardojbaez
Copy link
Owner

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 (7f8c23b), but has not been released yet.

@sanjaylive
Copy link

screen shot 2018-01-04 at 4 11 07 pm

@gerardojbaez
Copy link
Owner

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
Copy link

I've the same issue in laravel 5.5.

@sanjaylive
Copy link

sanjaylive commented Jan 26, 2018 via email

@siarheipashkevich
Copy link

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

@josteph
Copy link

josteph commented Feb 23, 2018

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

@oanhnn
Copy link

oanhnn commented Feb 23, 2018

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

@josteph
Copy link

josteph commented Feb 23, 2018

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

@oanhnn
Copy link

oanhnn commented Feb 23, 2018

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

@gerardojbaez
Copy link
Owner

gerardojbaez commented Feb 23, 2018

@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
Copy link

oanhnn commented Feb 23, 2018

@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
Copy link
Owner

gerardojbaez commented Feb 23, 2018

@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
Copy link
Owner

The release is out! v2.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants