Skip to content

remove owned param#52

Merged
laurenleach merged 6 commits intomainfrom
lauren/remove-owned-param
Aug 18, 2025
Merged

remove owned param#52
laurenleach merged 6 commits intomainfrom
lauren/remove-owned-param

Conversation

@laurenleach
Copy link
Copy Markdown
Contributor

Description

  • Bug fix
  • New feature

Useful links:

cursor[bot]

This comment was marked as outdated.

@laurenleach laurenleach force-pushed the lauren/remove-owned-param branch from fb91e4e to 2bc6543 Compare August 16, 2025 14:34

apiURL, _ := url.Parse("/api/v4/groups")
query := apiURL.Query()
query.Set("owned", "true")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if you remove this param we should add defensive code on listing pending invited users, since the list is unavailable if you are not owner/admin of the project/group in which you are listing users

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@laurenleach laurenleach merged commit d8e8e7c into main Aug 18, 2025
5 checks passed
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.

2 participants