Explorar el Código

Add 'setting-changed' tracking when user changes user-wide settings

GitOrigin-RevId: 04ed66dcdff7fa04e783fe4028505d981a39b37b
M Fahru hace 3 años
padre
commit
ae5a08e680

+ 6 - 0
services/web/frontend/js/features/editor-left-menu/hooks/use-save-user-settings.tsx

@@ -1,4 +1,5 @@
 import useScopeValue from '../../../shared/hooks/use-scope-value'
 import useScopeValue from '../../../shared/hooks/use-scope-value'
+import { sendMB } from '../../../infrastructure/event-tracking'
 import { saveUserSettings } from '../utils/api'
 import { saveUserSettings } from '../utils/api'
 import type { UserSettingsScope } from '../utils/api'
 import type { UserSettingsScope } from '../utils/api'
 
 
@@ -12,6 +13,11 @@ export default function useSaveUserSettings() {
   ) => {
   ) => {
     const currentSetting = userSettingsScope[key]
     const currentSetting = userSettingsScope[key]
 
 
+    sendMB('setting-changed', {
+      changedSetting: key,
+      changedSettingVal: newSetting,
+    })
+
     if (currentSetting !== newSetting) {
     if (currentSetting !== newSetting) {
       setUserSettingsScope({ ...userSettingsScope, [key]: newSetting })
       setUserSettingsScope({ ...userSettingsScope, [key]: newSetting })
       saveUserSettings({ [key]: newSetting })
       saveUserSettings({ [key]: newSetting })

+ 6 - 0
services/web/frontend/js/features/editor-left-menu/hooks/use-set-spell-check-language.tsx

@@ -1,4 +1,5 @@
 import { useCallback } from 'react'
 import { useCallback } from 'react'
+import { sendMB } from '../../../infrastructure/event-tracking'
 import { useProjectContext } from '../../../shared/context/project-context'
 import { useProjectContext } from '../../../shared/context/project-context'
 import useScopeValue from '../../../shared/hooks/use-scope-value'
 import useScopeValue from '../../../shared/hooks/use-scope-value'
 import { saveProjectSettings, saveUserSettings } from '../utils/api'
 import { saveProjectSettings, saveUserSettings } from '../utils/api'
@@ -11,6 +12,11 @@ export default function useSetSpellCheckLanguage() {
   const setSpellCheckLanguage = useCallback(
   const setSpellCheckLanguage = useCallback(
     (spellCheckLanguage: string) => {
     (spellCheckLanguage: string) => {
       if (spellCheckLanguageScope) {
       if (spellCheckLanguageScope) {
+        sendMB('setting-changed', {
+          changedSetting: 'spellCheckLanguage',
+          changedSettingVal: spellCheckLanguage,
+        })
+
         setSpellCheckLanguageScope(spellCheckLanguage)
         setSpellCheckLanguageScope(spellCheckLanguage)
 
 
         // save to both project setting and user setting
         // save to both project setting and user setting