Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rnelson01
Copy link

added docs example get all group members + access level

Changes

Documentation and testing

Please consider whether this PR needs documentation and tests. This is not required, but highly appreciated:

added docs example get all group members + access level
Copy link
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution @rnelson01!

The commit message will need to be changed to pass our CI, see our contribution guidelines if you need help. I just have one additional question as well.


members = group.members_all.list(get_all=True)

List all groups and all group member with access level::
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feels like it's covering a bit more than just the group members API and almost like a small cookbook example. WDYT about narrowing it down to just the API calls for members, or moving this example to a separate documentation page for longer examples?

@github-actions
Copy link

This Pull Request (PR) was marked stale because it has been open 90 days with no activity. Please remove the stale label or comment on this PR. Otherwise, it will be closed in 15 days.

@github-actions github-actions bot added the stale label Jun 26, 2025
@github-actions
Copy link

This PR was closed because it has been marked stale for 15 days with no activity. If this PR is still valid, please re-open.

@github-actions github-actions bot closed this Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants