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

Add pagination for GET /api/v1/jobs #4979

Merged
merged 2 commits into from
Jan 10, 2023
Merged

Add pagination for GET /api/v1/jobs #4979

merged 2 commits into from
Jan 10, 2023

Conversation

kraih
Copy link
Member

@kraih kraih commented Jan 10, 2023

Cleaned up version of #4964.

Progress: https://progress.opensuse.org/issues/121102

@Martchus
Copy link
Contributor

It would still make sense to extend the commit message a little bit. Otherwise it LGTM.

@kraih
Copy link
Member Author

kraih commented Jan 10, 2023

It would still make sense to extend the commit message a little bit. Otherwise it LGTM.

Editing someone else's commit feels weird. Not sure what the polite thing to do is here. 😬

@codecov
Copy link

codecov bot commented Jan 10, 2023

Codecov Report

Merging #4979 (fb1ffb6) into master (88fd8f3) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head fb1ffb6 differs from pull request most recent head 30b0222. Consider uploading reports for the commit 30b0222 to get more accurate results

@@            Coverage Diff             @@
##           master    #4979      +/-   ##
==========================================
- Coverage   98.17%   98.17%   -0.01%     
==========================================
  Files         379      379              
  Lines       35430    35456      +26     
==========================================
+ Hits        34785    34809      +24     
- Misses        645      647       +2     
Impacted Files Coverage Δ
lib/OpenQA/Schema/ResultSet/Jobs.pm 100.00% <100.00%> (ø)
lib/OpenQA/WebAPI/Controller/API/V1/Job.pm 96.80% <100.00%> (+0.05%) ⬆️
t/api/04-jobs.t 100.00% <100.00%> (ø)
lib/OpenQA/Worker/WebUIConnection.pm 94.58% <0.00%> (-0.99%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Similar pagination implementation to `/api/v1/bugs`.

Progress: https://progress.opensuse.org/issues/121102
@kraih
Copy link
Member Author

kraih commented Jan 10, 2023

Ok, commit message updated.

okurz
okurz previously requested changes Jan 10, 2023
t/api/04-jobs.t Outdated Show resolved Hide resolved
@mergify mergify bot merged commit a89d415 into master Jan 10, 2023
@Martchus Martchus deleted the jobs_pagination branch January 10, 2023 17:07
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

5 participants