pagure

#2494 Make the API returns only open issues by default.
Merged by pingou. Opened by cverna.
cverna/pagure fix_2468  into  master

Download 2494.patch

Signed-off-by: Clement Verna cverna@tutanota.com

FIx for #2468

rebased

Does it make sense to adjust one of these to return the full list?

wow, ?status=closed was not returning anything before, gloups :s

One question, otherwise looks good

It used to return the full list because there was no Closed status issues in the FULL_ISSUE_LIST. I have change on issue status to Closed so that we can test the case below.

Yes but would it make sense for one of the two tests we have here to include all tickets, open and closed?

There is already one test for that with ?status=All https://pagure.io/pagure/blob/master/f/tests/test_pagure_flask_api_issue.py#_1065

rebased

rebased

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.