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

[IOPAE-1680] backoffice auth check create services #1149

Merged
merged 6 commits into from
Feb 6, 2025

Conversation

leopoldo94
Copy link
Contributor

List of Changes

added check on active group for create service
added unit test

Motivation and Context

the necessity to check if the group, on which the user want to create the service, is active

How Has This Been Tested?

Unit test

Screenshots (if appropriate):

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@leopoldo94 leopoldo94 requested a review from a team as a code owner February 5, 2025 15:45
Copy link

changeset-bot bot commented Feb 5, 2025

🦋 Changeset detected

Latest commit: db74b03

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
io-services-cms-backoffice Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for App Backend (./apps/app-backend)

Status Category Percentage Covered / Total
🔵 Lines 81.98% 974 / 1188
🔵 Statements 81.98% 974 / 1188
🔵 Functions 72.5% 29 / 40
🔵 Branches 94.73% 126 / 133
File CoverageNo changed files found.
Generated in workflow #3233

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for Backoffice Frontend (./apps/backoffice)

Status Category Percentage Covered / Total
🔵 Lines 23.07% 2867 / 12427
🔵 Statements 23.07% 2867 / 12427
🔵 Functions 29.48% 74 / 251
🔵 Branches 65.39% 172 / 263
File CoverageNo changed files found.
Generated in workflow #3233

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for Backoffice Backend (./apps/backoffice)

Status Category Percentage Covered / Total
🔵 Lines 66.87% 4237 / 6336
🔵 Statements 66.87% 4237 / 6336
🔵 Functions 64% 128 / 200
🔵 Branches 82.79% 563 / 680
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
apps/backoffice/src/app/api/services/route.ts 91.07% 93.33% 100% 91.07% 107-121
apps/backoffice/src/lib/be/authz.ts 100% 100% 100% 100%
Generated in workflow #3233

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for Webapp (./apps/io-services-cms-webapp)

Status Category Percentage Covered / Total
🔵 Lines 90.34% 8582 / 9499
🔵 Statements 90.34% 8582 / 9499
🔵 Functions 88.7% 267 / 301
🔵 Branches 91.09% 910 / 999
File CoverageNo changed files found.
Generated in workflow #3233

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for External Clients (./packages/external-clients)

Status Category Percentage Covered / Total
🔵 Lines 88.65% 1024 / 1155
🔵 Statements 88.65% 1024 / 1155
🔵 Functions 79.16% 38 / 48
🔵 Branches 95.14% 98 / 103
File CoverageNo changed files found.
Generated in workflow #3233

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Coverage Report for Models (./packages/io-services-cms-models)

Status Category Percentage Covered / Total
🔵 Lines 85.99% 2044 / 2377
🔵 Statements 85.99% 2044 / 2377
🔵 Functions 64.89% 61 / 94
🔵 Branches 96.68% 175 / 181
File CoverageNo changed files found.
Generated in workflow #3233

Copy link

sonarqubecloud bot commented Feb 5, 2025

@leopoldo94 leopoldo94 added this pull request to the merge queue Feb 6, 2025
Merged via the queue into master with commit 5124721 Feb 6, 2025
6 checks passed
@leopoldo94 leopoldo94 deleted the IOPAE-1680-backoffice-auth-check-create-services branch February 6, 2025 11:33
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.

2 participants