Skip to content

fix: share and assign team issue#7687

Open
luka-nextcloud wants to merge 1 commit intomainfrom
fix-share-and-assign-team
Open

fix: share and assign team issue#7687
luka-nextcloud wants to merge 1 commit intomainfrom
fix-share-and-assign-team

Conversation

@luka-nextcloud
Copy link
Copy Markdown
Contributor

@luka-nextcloud luka-nextcloud commented Feb 26, 2026

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

@luka-nextcloud luka-nextcloud self-assigned this Feb 26, 2026
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Feb 26, 2026
@luka-nextcloud luka-nextcloud moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Feb 26, 2026
@luka-nextcloud luka-nextcloud force-pushed the fix-share-and-assign-team branch 2 times, most recently from 9771207 to d76c1bb Compare March 9, 2026 10:47
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 9, 2026

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 84632 was 84323 (+0.36%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

Copy link
Copy Markdown
Member

@grnd-alt grnd-alt left a comment

Choose a reason for hiding this comment

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

the failing phpunit tests say findAll called more than once, we're already querying for all acls in line 117, why not store and use again

@luka-nextcloud luka-nextcloud force-pushed the fix-share-and-assign-team branch from d76c1bb to 1dcf3a1 Compare April 1, 2026 14:40
Signed-off-by: Luka Trovic <luka@nextcloud.com>
@luka-nextcloud luka-nextcloud force-pushed the fix-share-and-assign-team branch from 1dcf3a1 to 42f1f7a Compare April 1, 2026 14:41
@luka-nextcloud luka-nextcloud requested a review from grnd-alt April 1, 2026 14:44
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 96079 was 93102 (+3.19%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 96070 was 93102 (+3.18%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

assignment of cards to circles (and groups)

2 participants