Skip to content

Address ultrareview findings on back-office tenant overview branch

970e544
Select commit
Loading
Failed to load commit list.
Open

Add back-office tenant overview with dashboard, accounts, users, billing events, and Stripe reconciliation #888

Address ultrareview findings on back-office tenant overview branch
970e544
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 10, 2026 in 1m 5s

Annotations

Check warning on line 20 in application/account/BackOffice/routes/users/index.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp6IzUDoXwVAk7Jl&open=AZ4FYp6IzUDoXwVAk7Jl&pullRequest=888

Check warning on line 124 in application/account/BackOffice/routes/accounts/-components/AccountActionsMenu.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp7GzUDoXwVAk7Jv&open=AZ4FYp7GzUDoXwVAk7Jv&pullRequest=888

Check warning on line 159 in application/account/BackOffice/routes/accounts/-components/AccountUsersTab.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use Array index in keys

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp7szUDoXwVAk7J2&open=AZ4FYp7szUDoXwVAk7J2&pullRequest=888

Check warning on line 34 in application/account/BackOffice/routes/users/-components/UserTenantsSection.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp5lzUDoXwVAk7Je&open=AZ4FYp5lzUDoXwVAk7Je&pullRequest=888

Check warning on line 76 in application/account/BackOffice/routes/accounts/-components/AccountsTable.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4HFMRmEnJDvK7FRQOb&open=AZ4HFMRmEnJDvK7FRQOb&pullRequest=888

Check warning on line 108 in application/account/BackOffice/routes/accounts/-components/AccountsTableRow.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp68zUDoXwVAk7Ju&open=AZ4FYp68zUDoXwVAk7Ju&pullRequest=888

Check failure on line 92 in application/account/BackOffice/routes/accounts/-components/AccountCurrentPlanCard.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp7azUDoXwVAk7Jy&open=AZ4FYp7azUDoXwVAk7Jy&pullRequest=888

Check warning on line 42 in application/account/BackOffice/routes/accounts/-components/AccountBillingEventRow.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp70zUDoXwVAk7J3&open=AZ4FYp70zUDoXwVAk7J3&pullRequest=888

Check warning on line 126 in application/account/Core/Features/Subscriptions/Domain/StripeEvent.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Method has 8 parameters, which is greater than the 7 authorized.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4OqXQHjt7_sTCI4Fug&open=AZ4OqXQHjt7_sTCI4Fug&pullRequest=888

Check warning on line 264 in application/account/Core/Features/Subscriptions/Shared/StripeEventReplayer.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of using this literal 'status' 4 times.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4OqXOAjt7_sTCI4Fuc&open=AZ4OqXOAjt7_sTCI4Fuc&pullRequest=888

Check warning on line 428 in application/account/Core/Features/Users/Domain/UserRepository.cs

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp-XzUDoXwVAk7KZ&open=AZ4FYp-XzUDoXwVAk7KZ&pullRequest=888

Check warning on line 45 in application/account/BackOffice/routes/billing-events/-components/BillingEventsTable.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4HFMS9EnJDvK7FRQOc&open=AZ4HFMS9EnJDvK7FRQOc&pullRequest=888

Check failure on line 72 in application/account/Core/Features/Subscriptions/Shared/ProcessPendingStripeEvents.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4RHbaXfu-4LUhW8lIx&open=AZ4RHbaXfu-4LUhW8lIx&pullRequest=888

Check warning on line 55 in application/account/BackOffice/routes/accounts/-components/AccountsToolbar.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4HFMO2EnJDvK7FRQOZ&open=AZ4HFMO2EnJDvK7FRQOZ&pullRequest=888

Check warning on line 36 in application/account/BackOffice/routes/accounts/index.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp8QzUDoXwVAk7KB&open=AZ4FYp8QzUDoXwVAk7KB&pullRequest=888

Check warning on line 20 in application/account/BackOffice/routes/billing-events/index.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp81zUDoXwVAk7KK&open=AZ4FYp81zUDoXwVAk7KK&pullRequest=888

Check warning on line 91 in application/account/Core/Features/Subscriptions/Domain/BillingEvent.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Method has 14 parameters, which is greater than the 7 authorized.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp-8zUDoXwVAk7Kg&open=AZ4FYp-8zUDoXwVAk7Kg&pullRequest=888

Check warning on line 59 in application/account/BackOffice/routes/-components/DashboardMrrTrendCard.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp8ZzUDoXwVAk7KD&open=AZ4FYp8ZzUDoXwVAk7KD&pullRequest=888

Check warning on line 114 in application/account/BackOffice/routes/accounts/-components/AccountBillingEventsSection.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp6azUDoXwVAk7Jp&open=AZ4FYp6azUDoXwVAk7Jp&pullRequest=888

Check warning on line 48 in application/account/Core/Features/BackOffice/Dashboard/Queries/GetDashboardTrends.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer the generic overload 'System.Enum.IsDefined<TEnum>(TEnum)' instead of 'System.Enum.IsDefined(System.Type, object)'

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp_azUDoXwVAk7Kj&open=AZ4FYp_azUDoXwVAk7Kj&pullRequest=888

Check warning on line 124 in application/account/Core/Features/BackOffice/BillingEvents/Queries/GetBackOfficeBillingEvents.cs

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp_kzUDoXwVAk7Kl&open=AZ4FYp_kzUDoXwVAk7Kl&pullRequest=888

Check warning on line 33 in application/account/BackOffice/routes/accounts/index.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp8QzUDoXwVAk7KA&open=AZ4FYp8QzUDoXwVAk7KA&pullRequest=888

Check warning on line 50 in application/account/BackOffice/routes/billing-events/-components/BillingEventsTableRow.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition.

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp89zUDoXwVAk7KN&open=AZ4FYp89zUDoXwVAk7KN&pullRequest=888

Check warning on line 49 in application/account/Core/Features/BackOffice/Dashboard/Queries/GetDashboardTrends.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer the generic overload 'System.Enum.IsDefined<TEnum>(TEnum)' instead of 'System.Enum.IsDefined(System.Type, object)'

See more on https://sonarcloud.io/project/issues?id=PlatformPlatform_platformplatform&issues=AZ4FYp_azUDoXwVAk7Kk&open=AZ4FYp_azUDoXwVAk7Kk&pullRequest=888

Check warning on line 53 in application/account/BackOffice/routes/users/-components/UserTenantsSection.tsx

See this annotation in the file changed.

@sonarqubecloud 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=AZ4FYp5lzUDoXwVAk7Jf&open=AZ4FYp5lzUDoXwVAk7Jf&pullRequest=888