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

No Pagination in Useradmin #19

Closed
phillipthelen opened this issue Mar 13, 2013 · 4 comments
Closed

No Pagination in Useradmin #19

phillipthelen opened this issue Mar 13, 2013 · 4 comments

Comments

@phillipthelen
Copy link
Contributor

The pagination for the useradmin seems to be broken. There are no prev/next buttons and when I set the current_page get value it also returns the first page.

The users are added through LDAP. (Theiß might be the source oft the bug? )

Looking at the source code I noticed Seahub isn't using the Django Pagination. Is there a specific reason for this?

@xiez
Copy link
Contributor

xiez commented Mar 14, 2013

What's your server version? Does your user number exceeds 25? 'cause the pagination in our public server works well.

Currently, we implement pagination by ourself. We will look into Django Pagination to see whether it fits our requirement.

On Mar 14, 2013, at 3:32 AM, Phillip Thelen notifications@github.com wrote:

The pagination for the useradmin seems to be broken. There are no prev/next buttons and when I set the current_page get value it also returns the first page.

Looking at the source code I noticed Seahub isn't using the Django Pagination. Is there a specific reason for this?


Reply to this email directly or view it on GitHub.

@phillipthelen
Copy link
Contributor Author

We are using the current 1.5.0 Server. There are more than 100 users (1 admin created during installation and the rest are ldap users). Maybe also should be noted: only two of the ldap users have already logged in.

@xiez
Copy link
Contributor

xiez commented Mar 15, 2013

Okay, we have reproduced this bug with ldap enabled, will be fixed shortly.

On Mar 14, 2013, at 8:17 PM, Phillip Thelen notifications@github.com wrote:

We are using the current 1.5.0 Server. There are more than 100 users (1 admin created during installation and the rest are ldap users). Maybe also should be noted: only two of the ldap users have already logged in.


Reply to this email directly or view it on GitHub.

@xiez
Copy link
Contributor

xiez commented Apr 19, 2013

Fixed in 1.6.0, you can download from http://seafile.com/en/download/.

@xiez xiez closed this as completed Apr 19, 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

No branches or pull requests

2 participants