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

add a path() function #21

Merged
merged 7 commits into from
Sep 22, 2014
Merged

add a path() function #21

merged 7 commits into from
Sep 22, 2014

Conversation

Mahmoudz
Copy link
Contributor

add a cdn::path() function to allow users upload files from the app dir
make cdn::asset() search for files in the public dir
update the documentation

mahmoud added 5 commits September 19, 2014 19:02
the difference between those 2 functions is
assets always starts from the public/ while asset takes a full path
starting from the app/
to affect this commit 6a0ed63 changes
rename preparePathAndCallUrlGenerator to generateUrl
and update this function parameter

related to commit #6a0ed63b22c20e3117c4e35c8b2f35e8926b4ccf
rename testPreparePathThrowsExceptionWhenEmptyParameter to
testUrlGeneratorThrowsException

related to commit #6a0ed63b22c20e3117c4e35c8b2f35e8926b4ccf
@Mahmoudz
Copy link
Contributor Author

re-factoring

mahmoud added 2 commits September 22, 2014 16:52
related to commit #6a0ed63b22c20e3117c4e35c8b2f35e8926b4ccf
related to commit #6a0ed63b22c20e3117c4e35c8b2f35e8926b4ccf
Mulkave added a commit that referenced this pull request Sep 22, 2014
@Mulkave Mulkave merged commit 2f1d8ae into develop Sep 22, 2014
@Mulkave Mulkave deleted the update/add-path-function branch September 22, 2014 14:21
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.

2 participants