Skip to content

NDA Approval Email - #2453

Merged
golanglemonade merged 3 commits into
mainfrom
feat-ndaapprovalnotification
Jun 4, 2026
Merged

NDA Approval Email#2453
golanglemonade merged 3 commits into
mainfrom
feat-ndaapprovalnotification

fix graphapi tests

dbf9075
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 4, 2026 in 1m 34s

Quality Gate failed

Failed conditions
5.5% Coverage on New Code (required ≥ 20%)
5.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Annotations

Check failure on line 120 in internal/ent/hooks/trustcenterndarequest.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 19 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ6Slt7fNq_NGq_VPrDN&open=AZ6Slt7fNq_NGq_VPrDN&pullRequest=2453

Check failure on line 101 in internal/ent/hooks/trustcenterndarequest.go

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "failed to send NDA approval request emails" 3 times. 

See more on https://sonarcloud.io/project/issues?id=theopenlane_core&issues=AZ6Slt7fNq_NGq_VPrDM&open=AZ6Slt7fNq_NGq_VPrDM&pullRequest=2453
View more details on SonarQubeCloud