From 886abe4e555e3d0a5ecf04aa828e254e2aee57b1 Mon Sep 17 00:00:00 2001 From: Niranjana Binoy <43930900+NiranjanaBinoy@users.noreply.github.com> Date: Thu, 12 Sep 2024 16:35:04 -0400 Subject: [PATCH] addressing review comments --- .../metametrics-data-deletion.ts | 10 +--------- shared/constants/metametrics.ts | 16 ++++++++-------- 2 files changed, 9 insertions(+), 17 deletions(-) diff --git a/app/scripts/controllers/metametrics-data-deletion/metametrics-data-deletion.ts b/app/scripts/controllers/metametrics-data-deletion/metametrics-data-deletion.ts index 811d450f0b77..8aad81c3721d 100644 --- a/app/scripts/controllers/metametrics-data-deletion/metametrics-data-deletion.ts +++ b/app/scripts/controllers/metametrics-data-deletion/metametrics-data-deletion.ts @@ -2,21 +2,13 @@ import { BaseController, RestrictedControllerMessenger, } from '@metamask/base-controller'; +import { PublicInterface } from '@metamask/utils'; import type { DataDeletionService } from '../../services/data-deletion-service'; import { DeleteRegulationStatus } from '../../../../shared/constants/metametrics'; // Unique name for the controller const controllerName = 'MetaMetricsDataDeletionController'; -/** - * Get a type representing the public interface of the given type. The - * returned type will have all public properties, but will omit private - * properties. - * - * @template Interface - The interface to return a public representation of. - */ -type PublicInterface = Pick; - /** * Timestamp at which regulation response is returned. */ diff --git a/shared/constants/metametrics.ts b/shared/constants/metametrics.ts index a8686dbd0a42..502f166c8e91 100644 --- a/shared/constants/metametrics.ts +++ b/shared/constants/metametrics.ts @@ -855,12 +855,12 @@ export enum MetaMetricsContextProp { * Mentioned here: https://docs.segmentapis.com/tag/Deletion-and-Suppression#operation/listRegulationsFromSource */ export enum DeleteRegulationStatus { - failed = 'FAILED', - finished = 'FINISHED', - initialized = 'INITIALIZED', - invalid = 'INVALID', - notSupported = 'NOT_SUPPORTED', - partialSuccess = 'PARTIAL_SUCCESS', - running = 'RUNNING', - unknown = 'UNKNOWN', + Failed = 'FAILED', + Finished = 'FINISHED', + Initialized = 'INITIALIZED', + Invalid = 'INVALID', + NotSupported = 'NOT_SUPPORTED', + PartialSuccess = 'PARTIAL_SUCCESS', + Running = 'RUNNING', + Unknown = 'UNKNOWN', }