Skip to content

[WPB-27705] Roll back: do *not* include collaborator apps in get-apps end-point. - #5402

Merged
fisx merged 4 commits into
developfrom
WPB-27705-roll-back_-do-_not_-include-collaborator-apps-in-get-apps-end-point
Jul 31, 2026
Merged

[WPB-27705] Roll back: do *not* include collaborator apps in get-apps end-point.#5402
fisx merged 4 commits into
developfrom
WPB-27705-roll-back_-do-_not_-include-collaborator-apps-in-get-apps-end-point

Conversation

@fisx

@fisx fisx commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-27705

There is GET /teams/:tid/apps and GET /teams/:tid/collaborators, and mixing the two concerns in these two end-points seems incoherent to me.

There is an ongoing discussion whether collaborators are team members (they interact with the team, but do you have to pay for their seats, even though the home team has already paid?), but for these two end-points there shouldn't be any disagreements. (... right?)

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 31, 2026
… "GET /teams/:tid/apps". (#5343)"

This partially reverts commits
3cb74c4,
632be41, but leaves in some drive-by
changes.
@fisx
fisx marked this pull request as ready for review July 31, 2026 10:50
@fisx
fisx requested review from a team as code owners July 31, 2026 10:50

@supersven supersven left a comment

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.

The why should be documented; in the commit message, changelog or PR description. This would help your colleagues answering possible questions regarding this when you're not around.

Other than that: Looks good to me 👍

@fisx
fisx merged commit f07b903 into develop Jul 31, 2026
8 checks passed
@fisx
fisx deleted the WPB-27705-roll-back_-do-_not_-include-collaborator-apps-in-get-apps-end-point branch July 31, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants