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

Projects directory listing only shows 10 projects #2540

Closed
punchagan opened this issue Feb 17, 2017 · 0 comments
Closed

Projects directory listing only shows 10 projects #2540

punchagan opened this issue Feb 17, 2017 · 0 comments
Assignees
Milestone

Comments

@punchagan
Copy link
Contributor

The project listing directory is hard coded to show only 10 projects. This limit seems to be in place due to some changes made in #1258 and may not be necessary any more.

@punchagan punchagan self-assigned this Feb 17, 2017
punchagan added a commit that referenced this issue Feb 17, 2017
The query for `partnership_more_link` is very slow, and adding a
`select_related` makes it quite a lot faster.  This looks like the only query
that is slow on the project listing page.

We add a `limit` parameter to specify the number of projects to be listed in
the directory, with the default value being 10.

Closes #2540
punchagan added a commit that referenced this issue Feb 17, 2017
And optimize check for multiple currencies in a project
punchagan added a commit that referenced this issue Feb 17, 2017
And refactor codelist_name function to use this.  These functions are called in
so many places, that caching them should bring significant benefits all over
the site.

RSR seems to already have memcached configured and it's a shame that we don't
use it anywhere except for thumbnails.  A lot of things can benefit from caching!
punchagan added a commit that referenced this issue Feb 17, 2017
punchagan added a commit that referenced this issue Feb 20, 2017
Added a simple performance test example for the project directory listing, but
the test cannot be run with the current version of Django.  It can be
un-skipped when Django is updated!
@punchagan punchagan added this to the 3.22 Chisinau milestone Feb 21, 2017
punchagan added a commit that referenced this issue Feb 23, 2017
The query for `partnership_more_link` is very slow, and adding a
`select_related` makes it quite a lot faster.  This looks like the only query
that is slow on the project listing page.

We add a `limit` parameter to specify the number of projects to be listed in
the directory, with the default value being 10.

Closes #2540
punchagan added a commit that referenced this issue Feb 23, 2017
And optimize check for multiple currencies in a project
punchagan added a commit that referenced this issue Feb 23, 2017
And refactor codelist_name function to use this.  These functions are called in
so many places, that caching them should bring significant benefits all over
the site.

RSR seems to already have memcached configured and it's a shame that we don't
use it anywhere except for thumbnails.  A lot of things can benefit from caching!
punchagan added a commit that referenced this issue Feb 23, 2017
punchagan added a commit that referenced this issue Feb 23, 2017
Added a simple performance test example for the project directory listing, but
the test cannot be run with the current version of Django.  It can be
un-skipped when Django is updated!
punchagan added a commit that referenced this issue Feb 23, 2017
And optimize check for multiple currencies in a project
punchagan added a commit that referenced this issue Feb 23, 2017
And refactor codelist_name function to use this.  These functions are called in
so many places, that caching them should bring significant benefits all over
the site.

RSR seems to already have memcached configured and it's a shame that we don't
use it anywhere except for thumbnails.  A lot of things can benefit from caching!
punchagan added a commit that referenced this issue Feb 23, 2017
punchagan added a commit that referenced this issue Feb 23, 2017
Added a simple performance test example for the project directory listing, but
the test cannot be run with the current version of Django.  It can be
un-skipped when Django is updated!
zzgvh added a commit that referenced this issue Mar 8, 2017
[#2540] Allow listing more projects in the directory
Code reviewed
@zzgvh zzgvh removed the Needs Review label Mar 8, 2017
@MichaelAkvo MichaelAkvo added this to RSR Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants