diff options
| -rw-r--r-- | src/components/moderation/ReportDialog/index.tsx | 23 | ||||
| -rw-r--r-- | src/logger/metrics.ts | 11 | ||||
| -rw-r--r-- | src/logger/types.ts | 1 |
3 files changed, 33 insertions, 2 deletions
diff --git a/src/components/moderation/ReportDialog/index.tsx b/src/components/moderation/ReportDialog/index.tsx index 7115324e6..291699380 100644 --- a/src/components/moderation/ReportDialog/index.tsx +++ b/src/components/moderation/ReportDialog/index.tsx @@ -8,7 +8,7 @@ import {useLingui} from '@lingui/react' import {wait} from '#/lib/async/wait' import {getLabelingServiceTitle} from '#/lib/moderation' import {sanitizeHandle} from '#/lib/strings/handles' -import {logger} from '#/logger' +import {Logger} from '#/logger' import {isNative} from '#/platform/detection' import {useMyLabelersQuery} from '#/state/queries/preferences' import {CharProgress} from '#/view/com/composer/char-progress/CharProgress' @@ -39,6 +39,8 @@ import {ReportOption, useReportOptions} from './utils/useReportOptions' export {useDialogControl as useReportDialogControl} from '#/components/Dialog' +const logger = Logger.create(Logger.Context.ReportDialog) + export function ReportDialog( props: Omit<ReportDialogProps, 'subject'> & { subject: ReportSubject @@ -48,8 +50,11 @@ export function ReportDialog( () => parseReportSubject(props.subject), [props.subject], ) + const onClose = React.useCallback(() => { + logger.metric('reportDialog:close', {}) + }, []) return ( - <Dialog.Outer control={props.control}> + <Dialog.Outer control={props.control} onClose={onClose}> <Dialog.Handle /> {subject ? <Inner {...props} subject={subject} /> : <Invalid />} </Dialog.Outer> @@ -137,6 +142,8 @@ function Inner(props: ReportDialogProps) { const onSubmit = React.useCallback(async () => { dispatch({type: 'clearError'}) + logger.info('submitting') + try { setPending(true) // wait at least 1s, make it feel substantial @@ -148,11 +155,17 @@ function Inner(props: ReportDialogProps) { }), ) setSuccess(true) + logger.metric('reportDialog:success', { + reason: state.selectedOption?.reason!, + labeler: state.selectedLabeler?.creator.handle!, + details: !!state.details, + }) // give time for user feedback setTimeout(() => { props.control.close() }, 1e3) } catch (e: any) { + logger.metric('reportDialog:failure', {}) logger.error(e, { source: 'ReportDialog', }) @@ -165,6 +178,12 @@ function Inner(props: ReportDialogProps) { } }, [_, submitReport, state, dispatch, props, setPending, setSuccess]) + React.useEffect(() => { + logger.metric('reportDialog:open', { + subjectType: props.subject.type, + }) + }, [props.subject]) + return ( <Dialog.ScrollableInner label={_(msg`Report dialog`)} diff --git a/src/logger/metrics.ts b/src/logger/metrics.ts index e3bd93314..66f79e4f6 100644 --- a/src/logger/metrics.ts +++ b/src/logger/metrics.ts @@ -305,4 +305,15 @@ export type MetricEvents = { 'progressGuide:hide': {} 'progressGuide:followDialog:open': {} + + 'reportDialog:open': { + subjectType: string + } + 'reportDialog:close': {} + 'reportDialog:success': { + reason: string + labeler: string + details: boolean + } + 'reportDialog:failure': {} } diff --git a/src/logger/types.ts b/src/logger/types.ts index 9110a8c6f..d14e21a9d 100644 --- a/src/logger/types.ts +++ b/src/logger/types.ts @@ -9,6 +9,7 @@ export enum LogContext { Notifications = 'notifications', ConversationAgent = 'conversation-agent', DMsAgent = 'dms-agent', + ReportDialog = 'report-dialog', /** * METRIC IS FOR INTERNAL USE ONLY, don't create any other loggers using this |