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

Skip to content

Visibility flag for teams (e.g. hide company wide teams) - #6182

Draft
kevinpapst wants to merge 8 commits into
mainfrom
private-teams
Draft

Visibility flag for teams (e.g. hide company wide teams)#6182
kevinpapst wants to merge 8 commits into
mainfrom
private-teams

Conversation

@kevinpapst

@kevinpapst kevinpapst commented Sep 13, 2026

Copy link
Copy Markdown
Member

This flag is not the same as the visibility in customer or projects!

Team visibility is for setups where e.g. company wide teams exist.
Right now certain reports show all members of all the teams where the user is a teamlead or even member of. But those global teams should not be taken into account when calculating a list of users and it should not be selectable in dropdowns.

When switching the default visibility to false, then a team is not shown in dropdowns anymore.
There are new helper functions to filter e.g. "visible teams" or all "visible team mates".

TODO

  • check all reports where this could be useful
  • document feature
  • THIS IS NOT A SECURITY RELATED FEATURE! This is for convenience only and meant to limit the length of lists

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I verified that my code applies to the guidelines (composer code-check)
  • I updated the documentation (see here)
  • I agree that this code is used in Kimai (see license)

@codecov

codecov Bot commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.18182% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.83%. Comparing base (d49324c) to head (55994f1).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/Repository/UserRepository.php 75.00% 2 Missing ⚠️
src/Entity/User.php 95.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #6182      +/-   ##
============================================
+ Coverage     89.81%   89.83%   +0.02%     
- Complexity    10214    10230      +16     
============================================
  Files           898      898              
  Lines         32467    32492      +25     
============================================
+ Hits          29159    29189      +30     
+ Misses         3308     3303       -5     
Files with missing lines Coverage Δ
src/Controller/TeamController.php 93.57% <100.00%> (+0.05%) ⬆️
src/Entity/Team.php 100.00% <100.00%> (ø)
src/Form/TeamEditForm.php 100.00% <100.00%> (ø)
src/Repository/Query/UserQuery.php 100.00% <100.00%> (ø)
src/Repository/TeamRepository.php 81.19% <100.00%> (+0.49%) ⬆️
src/Entity/User.php 96.42% <95.00%> (+0.13%) ⬆️
src/Repository/UserRepository.php 84.46% <75.00%> (+1.67%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant