Skip to content
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

API permission #4779

Merged
merged 5 commits into from Apr 13, 2024
Merged

API permission #4779

merged 5 commits into from Apr 13, 2024

Conversation

kevinpapst
Copy link
Member

Description

added permission "api_access" to allow limiting API access

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)

Copy link

codecov bot commented Apr 13, 2024

Codecov Report

Attention: Patch coverage is 94.73684% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 87.86%. Comparing base (345bb66) to head (16d27c6).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #4779   +/-   ##
=========================================
  Coverage     87.86%   87.86%           
- Complexity     8925     8933    +8     
=========================================
  Files           792      794    +2     
  Lines         29310    29325   +15     
=========================================
+ Hits          25752    25766   +14     
- Misses         3558     3559    +1     
Files Coverage Δ
src/API/ActionsController.php 100.00% <ø> (ø)
src/API/ActivityController.php 97.32% <ø> (ø)
...c/API/Authentication/AccessTokenSuccessHandler.php 100.00% <100.00%> (ø)
src/API/ConfigurationController.php 100.00% <100.00%> (ø)
src/API/CustomerController.php 99.00% <ø> (ø)
src/API/ProjectController.php 94.61% <ø> (ø)
src/API/StatusController.php 90.90% <100.00%> (ø)
src/API/TagController.php 96.00% <100.00%> (ø)
src/API/TeamController.php 100.00% <100.00%> (ø)
src/API/TimesheetController.php 94.71% <ø> (ø)
... and 2 more

@kevinpapst kevinpapst merged commit 9dc9c71 into main Apr 13, 2024
5 checks passed
@kevinpapst kevinpapst deleted the api-permission branch April 13, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

1 participant