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

feat: use platform property to evaluate type of user, update all refe… #7162

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

YounixM
Copy link
Member

@YounixM YounixM commented Feb 20, 2025

…rences

use platform property to evaluate type of user, update all references


Important

Refactor user type evaluation to use useGetTenantLicense hook across the codebase.

  • Behavior:
    • Replaces isCloudUser() and isEECloudUser() with useGetTenantLicense() hook across multiple components to determine user type.
    • Affects user type checks in PrivateRoute, App, LaunchChatSupport, AttributeCheckList, AppLayout, BillingContainer, GeneralSettings, Retention, DashboardsList, LogsError, NoLogs, ServiceMetricTable, ServiceTraceTable, SideNav, IntegrationDetailPage, IntegrationsList, MessagingQueues, SettingsPage, NoData, and utils.ts.
  • Hooks:
    • Adds useGetTenantLicense hook in useGetTenantLicense.ts to return isCloudUser and isEECloudUser based on activeLicenseV3.platform.
  • Enums:
    • Adds LicensePlatform enum to getActive.ts to define SELF_HOSTED and CLOUD platforms.

This description was created by Ellipsis for 7e7e8ad. It will automatically update as commits are pushed.

@github-actions github-actions bot added docs required enhancement New feature or request labels Feb 20, 2025
@YounixM YounixM force-pushed the feat/active-license-updates branch from 7e7e8ad to e6104f9 Compare February 20, 2025 09:04
@@ -3,6 +3,7 @@ import setLocalStorageApi from 'api/browser/localstorage/set';
import getOrgUser from 'api/user/getOrgUser';
import { LOCALSTORAGE } from 'constants/localStorage';
import ROUTES from 'constants/routes';
import { useGetTenantLicense } from 'hooks/useGetTenantLicense';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: useGetDeploymentLicense

@YounixM YounixM force-pushed the feat/active-license-updates branch from e6104f9 to af06d5a Compare February 20, 2025 11:01
@YounixM YounixM force-pushed the feat/active-license-updates branch from af06d5a to 4ae7e99 Compare February 20, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs required enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants