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

fix: Fetch repositories #32

Closed
wants to merge 3 commits into from
Closed

fix: Fetch repositories #32

wants to merge 3 commits into from

Conversation

saundefined
Copy link
Member

Currently, only 100 repositories are requested.
After adding doc-* repositories, there are more of them.

The repositories of the next pages are also loaded in the list and sorted by update

Currently only 100 repositories are requested.
After adding doc-*
repositories, there are more of them.

The repositories of the next
pages are also loaded in the list,
and sorted by update
@cmb69
Copy link
Member

cmb69 commented Dec 28, 2020

Ah, good catch! Still, I'm somewhat concerned about the performance; building that list is already quite slow.

Added load more button instead of autoload
@saundefined
Copy link
Member Author

@cmb69, thanks for the feedback!
What do you think of this change?

Instead of automatically loading the next repositories, add a button to load more.

@cmb69
Copy link
Member

cmb69 commented Dec 28, 2020

Oh, just noticed that this is about qa.php.net (I though it is about bugs.php.net). I haven't used that interface for years, but apparently performance is not really an issue there, so maybe just go with the first commit?

@saundefined
Copy link
Member Author

Sure, reverted :)

@saundefined
Copy link
Member Author

Feature removed in e4fa7d2

@saundefined saundefined closed this Apr 8, 2021
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