Skip to content

Com api#6153

Merged
dmetzner merged 1 commit intodevelopfrom
com-api
Feb 28, 2026
Merged

Com api#6153
dmetzner merged 1 commit intodevelopfrom
com-api

Conversation

@dmetzner
Copy link
Collaborator


Your checklist for this pull request

Please review the contributing guidelines and wiki pages of this repository.

  • Include the name and id of the Jira ticket in the PR’s title eg.: SHARE-666 The devils ticket
  • Choose the proper base branch (develop)
  • Confirm that the changes follow the project’s coding guidelines
  • Verify that the changes generate no warnings and errors
  • Verify to commit no other files than the intentionally changed ones
  • Include reasonable and readable tests verifying the added or changed behavior
  • Verify that all tests are passing (CI), if not please state the test cases in the section below
  • Perform a self-review of the changes
  • Stick to the project’s git workflow (rebase and squash your commits)
  • Verify that your changes do not have any conflicts with the base branch
  • Put your ticket into the Code Review section in Jira
  • Ask for a code reviewer
  • Check that your pull request has been successfully deployed to https://web-test-1.catrob.at/

Additional Description

TODO: Add additional information that is not in your commit-message here

@dmetzner dmetzner force-pushed the com-api branch 2 times, most recently from aea8841 to 321d32c Compare February 28, 2026 07:27
@codecov
Copy link

codecov bot commented Feb 28, 2026

Codecov Report

❌ Patch coverage is 26.85512% with 621 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.18%. Comparing base (c7bc005) to head (0569b53).
⚠️ Report is 13 commits behind head on develop.

Files with missing lines Patch % Lines
...i/OpenAPI/Server/Controller/CommentsController.php 15.04% 271 Missing ⚠️
src/Api/CommentsApi.php 54.28% 112 Missing ⚠️
...yRepository/User/Comment/UserCommentRepository.php 0.00% 92 Missing ⚠️
src/Api/OpenAPI/Server/Model/CommentResponse.php 0.00% 62 Missing ⚠️
...c/Api/OpenAPI/Server/Model/CommentListResponse.php 0.00% 20 Missing ⚠️
src/Api/OpenAPI/Server/Model/CommentUserInfo.php 0.00% 20 Missing ⚠️
...penAPI/Server/Model/CommentTranslationResponse.php 52.63% 18 Missing ⚠️
.../Api/OpenAPI/Server/Model/CommentCreateRequest.php 0.00% 14 Missing ⚠️
...ecurity/Authentication/FormLoginSuccessHandler.php 18.18% 9 Missing ⚠️
...plication/Controller/Project/ProjectController.php 50.00% 1 Missing ⚠️
... and 2 more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6153      +/-   ##
=============================================
- Coverage      51.95%   51.18%   -0.78%     
- Complexity      7962     8182     +220     
=============================================
  Files            732      739       +7     
  Lines          25558    26285     +727     
=============================================
+ Hits           13279    13453     +174     
- Misses         12279    12832     +553     
Flag Coverage Δ
behat 48.62% <14.48%> (-1.12%) ⬇️
phpunit 10.59% <15.66%> (+0.22%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@dmetzner dmetzner marked this pull request as ready for review February 28, 2026 19:52
@dmetzner dmetzner merged commit 3785530 into develop Feb 28, 2026
51 checks passed
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.

1 participant