reasoning message should not be clickable until it has a message

This commit is contained in:
Nate Kelley 2025-04-21 13:17:26 -06:00
parent 9924120267
commit 73a07afbd8
No known key found for this signature in database
GPG Key ID: FD90372AB8D98B4F
4 changed files with 33 additions and 23 deletions

View File

@ -12,5 +12,6 @@ export enum BusterOrganizationRole {
WORKSPACE_ADMIN = 'workspaceAdmin', WORKSPACE_ADMIN = 'workspaceAdmin',
DATA_ADMIN = 'dataAdmin', DATA_ADMIN = 'dataAdmin',
QUERIER = 'querier', QUERIER = 'querier',
RESTRICTED_QUERIER = 'restrictedQuerier' RESTRICTED_QUERIER = 'restrictedQuerier',
VIEWER = 'viewer'
} }

View File

@ -1,4 +1,4 @@
import type { BusterOrganization } from '../organizations'; import type { BusterOrganization, BusterOrganizationRole } from '../organizations';
import type { BusterPermissionUser } from '../permission'; import type { BusterPermissionUser } from '../permission';
import type { ShareAssetType } from '../share'; import type { ShareAssetType } from '../share';
@ -70,7 +70,7 @@ export interface OrganizationUser {
email: string; email: string;
name: string; name: string;
status: 'active' | 'inactive'; status: 'active' | 'inactive';
role: 'dataAdmin' | 'workspaceAdmin' | 'querier' | 'restrictedQuerier' | 'viewer'; role: BusterOrganizationRole;
datasets: OrganizationUserDataset[]; datasets: OrganizationUserDataset[];
} }

View File

@ -1,4 +1,8 @@
import { type BusterUser, type OrganizationUser } from '@/api/asset_interfaces'; import {
BusterOrganizationRole,
type BusterUser,
type OrganizationUser
} from '@/api/asset_interfaces';
import React from 'react'; import React from 'react';
import { Title, Text } from '@/components/ui/typography'; import { Title, Text } from '@/components/ui/typography';
import { import {
@ -41,10 +45,11 @@ export const UserDefaultAccess: React.FC<{
}; };
const accessOptions: SelectItem<OrganizationUser['role']>[] = [ const accessOptions: SelectItem<OrganizationUser['role']>[] = [
{ label: 'Data Admin', value: 'dataAdmin' }, { label: 'Data Admin', value: BusterOrganizationRole.DATA_ADMIN },
{ label: 'Workspace Admin', value: 'workspaceAdmin' }, { label: 'Workspace Admin', value: BusterOrganizationRole.WORKSPACE_ADMIN },
{ label: 'Querier', value: 'querier' }, { label: 'Querier', value: BusterOrganizationRole.QUERIER },
{ label: 'Restricted Querier', value: 'restrictedQuerier' } { label: 'Restricted Querier', value: BusterOrganizationRole.RESTRICTED_QUERIER },
{ label: 'Viewer', value: BusterOrganizationRole.VIEWER }
]; ];
const DefaultAccessCard = React.memo( const DefaultAccessCard = React.memo(
@ -74,21 +79,23 @@ const DefaultAccessCard = React.memo(
<CardContent className="!pt-0"> <CardContent className="!pt-0">
<div className="flex items-center justify-between"> <div className="flex items-center justify-between">
<Text variant="secondary">Access level</Text> <Text variant="secondary">Access level</Text>
<AppTooltip <div className="min-w-44">
title={ <AppTooltip
isDisabled title={
? userIsMe isDisabled
? 'You cannot change your own access' ? userIsMe
: 'Only admins can change access' ? 'You cannot change your own access'
: undefined : 'Only admins can change access'
}> : undefined
<Select }>
items={accessOptions} <Select
value={role} items={accessOptions}
onChange={onChange} value={role}
disabled={isDisabled} onChange={onChange}
/> disabled={isDisabled}
</AppTooltip> />
</AppTooltip>
</div>
</div> </div>
</CardContent> </CardContent>
</Card> </Card>

View File

@ -50,6 +50,8 @@ export const ChatResponseReasoning: React.FC<{
}, [lastMessageTitle, finalReasoningMessage, blackBoxMessage]); }, [lastMessageTitle, finalReasoningMessage, blackBoxMessage]);
const href = useMemo(() => { const href = useMemo(() => {
if (!messageId) return '';
if (isReasonginFileSelected) { if (isReasonginFileSelected) {
return createBusterRoute({ return createBusterRoute({
route: BusterRoutes.APP_CHAT_ID, route: BusterRoutes.APP_CHAT_ID,