Add feature flag system with rollouts, plan gating, and back-office admin #896
Quality Gate passed
Annotations
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfu52iU0xy3YjkVt&open=AZ4okfu52iU0xy3YjkVt&pullRequest=896
Check failure on line 68 in application/AppGateway/Middleware/AuthenticationCookieMiddleware.cs
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=PlatformPlatform_platformplatform&issues=AZ4okfyL2iU0xy3YjkV_&open=AZ4okfyL2iU0xy3YjkV_&pullRequest=896
Check warning on line 73 in application/account/Tests/Users/InviteUserTests.cs
sonarqubecloud / SonarCloud Code Analysis
Prefer the string comparison method overload of 'string.Contains(string)' that takes a 'StringComparison' enum value to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase'
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4sZT_iY4VEIn5Z5OIj&open=AZ4sZT_iY4VEIn5Z5OIj&pullRequest=896
Check warning on line 238 in application/account/Tests/ArchitectureTests/EndpointMetadataTests.cs
sonarqubecloud / SonarCloud Code Analysis
Do not use Enumerable methods on indexable collections. Instead use the collection directly.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4sZUBrY4VEIn5Z5OIm&open=AZ4sZUBrY4VEIn5Z5OIm&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfuH2iU0xy3YjkVh&open=AZ4okfuH2iU0xy3YjkVh&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Remove this commented out code.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfxh2iU0xy3YjkV8&open=AZ4okfxh2iU0xy3YjkV8&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Refactor this code to not use nested template literals.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4rgR5tzgvZm-J9yQz9&open=AZ4rgR5tzgvZm-J9yQz9&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4qODK4ZxHbP9IiX36E&open=AZ4qODK4ZxHbP9IiX36E&pullRequest=896
Check warning on line 155 in application/account/Workers/FeatureFlagDefinitionReconciler.cs
sonarqubecloud / SonarCloud Code Analysis
Evaluation of this argument may be expensive and unnecessary if logging is disabled
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfww2iU0xy3YjkV1&open=AZ4okfww2iU0xy3YjkV1&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Unexpected negated condition.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfta2iU0xy3YjkVY&open=AZ4okfta2iU0xy3YjkVY&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okftw2iU0xy3YjkVe&open=AZ4okftw2iU0xy3YjkVe&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfti2iU0xy3YjkVb&open=AZ4okfti2iU0xy3YjkVb&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfti2iU0xy3YjkVZ&open=AZ4okfti2iU0xy3YjkVZ&pullRequest=896
Check warning on line 49 in application/account/BackOffice/routes/feature-flags/$flagKey.tsx
sonarqubecloud / SonarCloud Code Analysis
The signature '(entries: any, params?: string | { error?: string | $ZodErrorMap<$ZodIssueInvalidValue<unknown>> | undefined; message?: string | undefined; } | undefined): ZodEnum<...>' of 'z.nativeEnum' is deprecated.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfuP2iU0xy3YjkVl&open=AZ4okfuP2iU0xy3YjkVl&pullRequest=896
Check warning on line 201 in application/account/BackOffice/routes/feature-flags/$flagKey.tsx
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfuP2iU0xy3YjkVo&open=AZ4okfuP2iU0xy3YjkVo&pullRequest=896
Check warning on line 251 in application/account/Tests/EmailAuthentication/StartEmailLoginTests.cs
sonarqubecloud / SonarCloud Code Analysis
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase'
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4sZUCdY4VEIn5Z5OIr&open=AZ4sZUCdY4VEIn5Z5OIr&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Prefer the string comparison method overload of 'string.Contains(string)' that takes a 'StringComparison' enum value to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase'
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfwO2iU0xy3YjkVw&open=AZ4okfwO2iU0xy3YjkVw&pullRequest=896
Check warning on line 37 in application/account/BackOffice/routes/accounts/-components/MrrCell.tsx
sonarqubecloud / SonarCloud Code Analysis
Extract this nested ternary operation into an independent statement.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfuW2iU0xy3YjkVp&open=AZ4okfuW2iU0xy3YjkVp&pullRequest=896
Check warning on line 193 in application/account/Workers/FeatureFlagDefinitionReconciler.cs
sonarqubecloud / SonarCloud Code Analysis
Evaluation of this argument may be expensive and unnecessary if logging is disabled
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4rgSEJzgvZm-J9yQ0A&open=AZ4rgSEJzgvZm-J9yQ0A&pullRequest=896
sonarqubecloud / SonarCloud Code Analysis
Loops should be simplified using the "Where" LINQ method
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfwE2iU0xy3YjkVv&open=AZ4okfwE2iU0xy3YjkVv&pullRequest=896
Check warning on line 33 in application/shared-webapp/ui/scripts/generateFeatureFlagArtifacts.mjs
sonarqubecloud / SonarCloud Code Analysis
Prefer `String#replaceAll()` over `String#replace()`.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfw52iU0xy3YjkV5&open=AZ4okfw52iU0xy3YjkV5&pullRequest=896
Check warning on line 212 in application/account/Workers/FeatureFlagDefinitionReconciler.cs
sonarqubecloud / SonarCloud Code Analysis
Evaluation of this argument may be expensive and unnecessary if logging is disabled
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4okfww2iU0xy3YjkV3&open=AZ4okfww2iU0xy3YjkV3&pullRequest=896
Check warning on line 67 in application/account/Tests/Users/InviteUserTests.cs
sonarqubecloud / SonarCloud Code Analysis
Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase'
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4sZT_iY4VEIn5Z5OIi&open=AZ4sZT_iY4VEIn5Z5OIi&pullRequest=896
Check failure on line 82 in application/account/Workers/FeatureFlagDefinitionReconciler.cs
sonarqubecloud / SonarCloud Code Analysis
Refactor this method to reduce its Cognitive Complexity from 28 to the 15 allowed.
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4rgSEJzgvZm-J9yQz_&open=AZ4rgSEJzgvZm-J9yQz_&pullRequest=896
Check warning on line 114 in application/account/Tests/Users/InviteUserTests.cs
sonarqubecloud / SonarCloud Code Analysis
Prefer the string comparison method overload of 'string.Contains(string)' that takes a 'StringComparison' enum value to perform a case-insensitive comparison, but keep in mind that this might cause subtle changes in behavior, so make sure to conduct thorough testing after applying the suggestion, or if culturally sensitive comparison is not required, consider using 'StringComparison.OrdinalIgnoreCase'
See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4sZT_iY4VEIn5Z5OIl&open=AZ4sZT_iY4VEIn5Z5OIl&pullRequest=896