Skip to content

Replace uses of Teams.owned_sites/1 with better functions where applicable #5397

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 19, 2025

Conversation

zoldar
Copy link
Contributor

@zoldar zoldar commented May 15, 2025

Changes

Small refactor of Teams.owned_sites/1 and its uses.

@zoldar zoldar force-pushed the cs-team-sites-list branch 2 times, most recently from 9fa754e to f33edba Compare May 15, 2025 12:14
@zoldar zoldar changed the title List all team sites in CS Replace uses of Teams.owned_sites/1 with better functions where applicable May 15, 2025
@zoldar zoldar requested a review from aerosol May 15, 2025 12:16
@aerosol
Copy link
Member

aerosol commented May 19, 2025

@zoldar maybe now that #5400 is merged, we can make this accept an optional limit?

@zoldar
Copy link
Contributor Author

zoldar commented May 19, 2025

@aerosol Sure.

@zoldar zoldar force-pushed the cs-team-sites-list branch from f33edba to 6004c68 Compare May 19, 2025 07:59
@zoldar zoldar added this pull request to the merge queue May 19, 2025
Merged via the queue into master with commit 6987ea8 May 19, 2025
8 checks passed
@zoldar zoldar deleted the cs-team-sites-list branch May 19, 2025 08:43
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