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 PRs #322

Merged
merged 1 commit into from
May 2, 2021
Merged

Fix Fetch PRs #322

merged 1 commit into from
May 2, 2021

Conversation

joomdonation
Copy link

Pull Request for Issue # .

Summary of Changes

I use patch tester for testing PR on Joomla 4 and found that it could not fetch all the PRs (only 5 pages, mean 100 PRs). I tried to debug code and it seems Link needs to be changed to link (l in lower case). I made that change and were able to fetch all PRs.

Testing Instructions

  1. Try to fetch PRs
  2. Before patch: Only latest 100 PRs are fetched. After patch, all PRs are fetched.

@richard67
Copy link

We should check if version 3 has the same problem.

@roland-d roland-d merged commit 8a1eb29 into joomla-extensions:master May 2, 2021
@roland-d
Copy link
Collaborator

roland-d commented May 2, 2021

@joomdonation Thank you.

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.

3 participants