pagure

#2231 Add an API to get group information, including membership
Merged by pingou. Opened by mprahl.
Unknown source master

Download 2231.patch

This PR is broken into two commits. The first commit adds an API to get a group's information, and the second commit moves the "list groups" API to pagure.api.group.

rebased

rebased

rebased

1 new commit added

  • Remove usage of 'assert_called_once' in test because pinned version of mock doesn't support it

I don't think we expose (nor should expose) email addresses to non-authenticated requests

Cf: https://pagure.io/api/0/pagure/pull-request/2231

This is where the decorator about optional login in the API is helpful :)

1 new commit added

  • Add optional authentication to the groups API

@pingou, thanks for the review! My latest commit should address your comment.

pretty please pagure-ci rebuild

There are a few issues with jenkins currently:

  • one is a dependency issue fixed in: https://pagure.io/pagure/pull-request/2234
  • another is an actual problem in the test suite, fixed in the third commit of: https://pagure.io/pagure/pull-request/2233#commit_list 30440f0

rebased

Ok let's merge your PR first and then hopefully tests will be back on working with #2233 :)

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.