Skip to content

Sl 249 fix campaigns #103

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.

Sign up for GitHub

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

Merged
GabrielFcGoncalves merged 4 commits into from
Mar 22, 2026

campaigns work again

c5d97ce
Select commit
Loading
Failed to load commit list.
Merged

Sl 249 fix campaigns #103

campaigns work again
c5d97ce
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 22, 2026 in 57s

Quality Gate failed

Failed conditions
39.8% Coverage on New Code (required ≥ 80%)
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 183 in api/src/tasks/scheduler.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant Exception class; it derives from another which is already caught. 

See more on https://sonarcloud.io/project/issues?id=PEI-SecureLearning_core&issues=AZ0S2AnZ3JYEkI940TNd&open=AZ0S2AnZ3JYEkI940TNd&pullRequest=103

Check warning on line 38 in api/src/services/keycloak_client/base_handler.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this code to not construct the URL's path from user-controlled data. 

See more on https://sonarcloud.io/project/issues?id=PEI-SecureLearning_core&issues=AZ0S2Aky3JYEkI940TNc&open=AZ0S2Aky3JYEkI940TNc&pullRequest=103

Check failure on line 141 in api/src/services/templates.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Template not found" 4 times. 

See more on https://sonarcloud.io/project/issues?id=PEI-SecureLearning_core&issues=AZ0WUOi2EnrKnygj_VBR&open=AZ0WUOi2EnrKnygj_VBR&pullRequest=103

Check failure on line 106 in api/src/services/templates.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "$exists" 3 times. 

See more on https://sonarcloud.io/project/issues?id=PEI-SecureLearning_core&issues=AZ0WUOi2EnrKnygj_VBS&open=AZ0WUOi2EnrKnygj_VBS&pullRequest=103

Check failure on line 135 in api/src/services/templates.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Invalid template id" 4 times. 

See more on https://sonarcloud.io/project/issues?id=PEI-SecureLearning_core&issues=AZ0WUOi2EnrKnygj_VBQ&open=AZ0WUOi2EnrKnygj_VBQ&pullRequest=103
View more details on SonarQubeCloud