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

New Feature Request: can laradrop support paginator? #10

Open
cnweibo opened this issue Oct 14, 2016 · 4 comments
Open

New Feature Request: can laradrop support paginator? #10

cnweibo opened this issue Oct 14, 2016 · 4 comments

Comments

@cnweibo
Copy link

cnweibo commented Oct 14, 2016

Hi jasekz,
I will have many assets files managed by laradrop, can laradrop support paginator feature?
thanks~!

@jasekz
Copy link
Owner

jasekz commented Oct 14, 2016

It's not supported out of the box, and because it uses ajax to fetch data, the Laravel paginator probably won't work. At this time, you'd need to override or modify the laradrop.js file to accomplish this.

@cnweibo
Copy link
Author

cnweibo commented Oct 16, 2016

It seems the main issue pagainator is not supported is caused by laradop use serivice such as file->get, which will retrieve decendant and siblings. If we return resource directly, it will be easily implemented with laravel paginate(10) code.
The front end side paginator works pretty well combined with other resources fetched, currently, the only exception is assets managed can not due to above reason.
It will be a little hard to implement under current laradrop laravel side architecture for me.

But it really is important since there will be many files uploaded to server.

thanks~!

@cnweibo
Copy link
Author

cnweibo commented Oct 16, 2016

Currently, i workaround this by :
attach the laradrop_files with another model and create pagination via the attached model

@jasekz
Copy link
Owner

jasekz commented Oct 16, 2016

Thanks, @cnweibo. I made this issue into a 'feature request' and will look into it as soon as I can.

@jasekz jasekz closed this as completed Oct 16, 2016
@jasekz jasekz changed the title can laradrop support paginator? New Feature Request: can laradrop support paginator? Oct 16, 2016
@jasekz jasekz reopened this Oct 16, 2016
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

2 participants