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

Wrong Composer Install #11

Closed
KerryJones opened this issue Oct 15, 2014 · 7 comments
Closed

Wrong Composer Install #11

KerryJones opened this issue Oct 15, 2014 · 7 comments

Comments

@KerryJones
Copy link

In the composer file, "infusionsoft/php-sdk" does not work, but "infusionsoft/php-isdk" does.

@mfairch
Copy link
Contributor

mfairch commented Oct 15, 2014

What are you putting in your composer file?
"infusionsoft/php-sdk" does work.
https://packagist.org/packages/infusionsoft/php-sdk

@KerryJones
Copy link
Author

https://www.dropbox.com/s/w43wa3wojwq7zpw/Screenshot%202014-10-14%2019.48.23.png?dl=0

You can see what I did in the screenshot.

Did not work:
{
"require": {
"infusionsoft/php-sdk": "~1.0"
}
}

Worked:
{
"require": {
"infusionsoft/php-isdk": "~1.0"
}
}

On 10/14/2014 19:45, Michael Fairchild wrote:

What are you putting in your composer file?
"infusionsoft/php-sdk" does work.
https://packagist.org/packages/infusionsoft/php-sdk


Reply to this email directly or view it on GitHub
#11 (comment).

@mfairch
Copy link
Contributor

mfairch commented Oct 15, 2014

Use "dev-master". We need to tag a new release.

@KerryJones
Copy link
Author

Right now I was going to use isdk, is that the wrong one? Is dev-master
of "sdk" better?

-Kerry

On 10/14/2014 19:52, Michael Fairchild wrote:

Use "dev-master". We need to tag a new release.


Reply to this email directly or view it on GitHub
#11 (comment).

@mfairch
Copy link
Contributor

mfairch commented Oct 15, 2014

Use this one. It's our rewrite that will have all new features added to it in long run.

@KerryJones
Copy link
Author

Got it -- works. It would be helpful to update the README for others.

-Kerry

On 10/14/2014 19:59, Michael Fairchild wrote:

Use this one. It's our rewrite that will have all new features added
to it in long run.


Reply to this email directly or view it on GitHub
#11 (comment).

@andrewryno
Copy link
Contributor

Updated the README to make that a little more clear. Thanks!

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

No branches or pull requests

3 participants