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

Readme fix #4

Merged
merged 4 commits into from Dec 16, 2013
Merged

Readme fix #4

merged 4 commits into from Dec 16, 2013

Conversation

JJK801
Copy link
Contributor

@JJK801 JJK801 commented Nov 19, 2013

Hi,

We implemented your useful plugin in our architecture, but we got some trouble due to documentation mistakes and lacks.

This PR provides all needed documentation to make it work quick & easy.

Thanks,

Jérémy


### Project scope installation

`/!\` This way needs you perform pluggin installation BEFORE to add packages links using S3

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plugin

this way needs ? (tu veux dire quoi par là ?)

Copy link
Contributor Author

@JJK801 JJK801 Nov 20, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is wrong (i understood that when making the Composer's PR), it will be erased

@omansour
Copy link

omansour commented Nov 19, 2013

👍

naderman added a commit that referenced this issue Dec 16, 2013
@naderman naderman merged commit 8036e5d into naderman:master Dec 16, 2013
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.

None yet

3 participants