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 ng-file-upload handler #39

Merged
merged 6 commits into from
May 30, 2018

Conversation

L3o-pold
Copy link
Contributor

* @return bool
*/
public static function canBeUsedForRequest(Request $request) {
return $request->has(static::KEY_CHUNK_NUMBER) && $request->has(static::KEY_TOTAL_SIZE)
Copy link
Owner

Choose a reason for hiding this comment

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

&& $request->has(static::KEY_TOTAL_SIZE) to a new line please :)

@pionl
Copy link
Owner

pionl commented May 30, 2018

Hi, thank you! Looks great. 👏

Could you rename ChunksInRequestNgFileUploadHandler to NgFileUploadHandler. I will rename the SimpleUploader too in next major release.

Do you have time to add unit test?

Copy link
Owner

@pionl pionl left a comment

Choose a reason for hiding this comment

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

Thank you 👍

@pionl
Copy link
Owner

pionl commented May 30, 2018

Creat job @L3o-pold ! Looks pretty sweat.

@pionl pionl merged commit 85fec50 into pionl:master May 30, 2018
@L3o-pold L3o-pold deleted the feature/ng-file-upload-handler branch May 30, 2018 19:13
@L3o-pold L3o-pold restored the feature/ng-file-upload-handler branch May 30, 2018 19:13
@pionl
Copy link
Owner

pionl commented May 30, 2018

New version out: https://github.com/pionl/laravel-chunk-upload/releases/tag/v1.2.5

I did test the dropzone/resumablejs/jquery. Unfortunately I can't test the NgFileUpload. I believe it's working 👍

@L3o-pold
Copy link
Contributor Author

Thank you for your fast response.
Yes of course, it's implemented in one of our project.

Tomorrow, I will add a small wiki page

@L3o-pold L3o-pold deleted the feature/ng-file-upload-handler branch May 31, 2018 08:56
@L3o-pold L3o-pold restored the feature/ng-file-upload-handler branch May 31, 2018 09:07
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