Skip to content

[BannerController] Add missing ProducesResponseType for Status403Forbidden#3842

Merged
imnasnainaec merged 2 commits intomasterfrom
banner-controller
Jun 4, 2025
Merged

[BannerController] Add missing ProducesResponseType for Status403Forbidden#3842
imnasnainaec merged 2 commits intomasterfrom
banner-controller

Conversation

@imnasnainaec
Copy link
Collaborator

@imnasnainaec imnasnainaec commented Jun 4, 2025

This change is Reviewable

@imnasnainaec imnasnainaec self-assigned this Jun 4, 2025
@imnasnainaec imnasnainaec added backend maintenance Issue that makes it difficult to maintain the software or to upgrade installations post-release. labels Jun 4, 2025
@imnasnainaec imnasnainaec changed the title [BannerController] Add missing [ProducesResponseType(StatusCodes.Status403Forbidden)] [BannerController] Add missing ProducesResponseType for Status403Forbidden Jun 4, 2025
@codecov
Copy link

codecov bot commented Jun 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.02%. Comparing base (ef8ad5b) to head (2d9bedd).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3842   +/-   ##
=======================================
  Coverage   73.02%   73.02%           
=======================================
  Files         286      286           
  Lines       10758    10758           
  Branches     1340     1340           
=======================================
  Hits         7856     7856           
  Misses       2502     2502           
  Partials      400      400           
Flag Coverage Δ
backend 82.15% <ø> (ø)
frontend 65.87% <ø> (ø)

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.

Copy link

@megahirt megahirt left a comment

Choose a reason for hiding this comment

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

LGTM

@imnasnainaec imnasnainaec merged commit 919afe0 into master Jun 4, 2025
14 checks passed
@imnasnainaec imnasnainaec deleted the banner-controller branch June 4, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend maintenance Issue that makes it difficult to maintain the software or to upgrade installations post-release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants