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

GPL-568 increase max results limit for endpoint #77

Merged
merged 2 commits into from
Aug 3, 2020

Conversation

KatyTaylor
Copy link
Contributor

Set pagination limit so that requests to this endpoint can put ?max_results=10000 and it will work

  • real reason is because for GPL-568, in lighthouse-ui Imports page we want to return all imports at once (~350 records at the moment)
  • previous limit was 50, as that is the default
  • see Eve documentation

…esults=10000 and it will work

- real reason is because for GPL-568, in lighthouse-ui Imports page we want to return all imports at once (~350 records at the moment)
- previous limit was 50, as that is the default
- see Eve documentation
@codecov
Copy link

codecov bot commented Aug 3, 2020

Codecov Report

Merging #77 into develop will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #77      +/-   ##
===========================================
+ Coverage    83.29%   83.33%   +0.03%     
===========================================
  Files           13       13              
  Lines          455      456       +1     
  Branches        53       53              
===========================================
+ Hits           379      380       +1     
  Misses          70       70              
  Partials         6        6              
Impacted Files Coverage Δ
lighthouse/config/defaults.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8dfd13...b4080a1. Read the comment docs.

@KatyTaylor KatyTaylor merged commit c4a441e into develop Aug 3, 2020
@KatyTaylor KatyTaylor deleted the GPL-568-increase-endpoint-max-results branch August 3, 2020 10:56
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.

None yet

3 participants