Skip to content

Commit

Permalink
fix(react) фикс тайпингов для deps массивов (#36)
Browse files Browse the repository at this point in the history
Co-authored-by: Aleksey Chepurko <[email protected]>
  • Loading branch information
AlekseyChepurko and Aleksey Chepurko authored Nov 27, 2023
1 parent 3e36bd9 commit 28f8dee
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 8 deletions.
4 changes: 2 additions & 2 deletions packages/react/src/usePageAnalytics/useModalPageAnalytics.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { CurrentStateStorage } from '@vkontakte/mini-apps-analytics';
import { useLayoutEffect } from 'react';
import { DependencyList, useLayoutEffect } from 'react';

export const useModalPageAnalytics = (modalPageName: string | null, deps: string[] = []) => {
export const useModalPageAnalytics = (modalPageName: string | null, deps: DependencyList = []) => {
/**
* Очищаем ранее собранную информацию по странице и инициируем новый сбор данных для страницы
*/
Expand Down
8 changes: 4 additions & 4 deletions packages/react/src/usePageAnalytics/usePageAnalytics.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CurrentStateStorage } from '@vkontakte/mini-apps-analytics';
import { useContext, useEffect, useLayoutEffect, useRef } from 'react';
import { DependencyList, useContext, useEffect, useLayoutEffect, useRef } from 'react';

import { analyticsContext } from '../context';
import { useModalPageAnalytics } from './useModalPageAnalytics';
Expand Down Expand Up @@ -30,17 +30,17 @@ export type UsePageAnalyticsParams = {
* Пример использования. Страница с различными табами навигации: панель остается неизменной, но рендерится новая страница
*
*/
pageDeps?: string[];
pageDeps?: DependencyList;
/**
* Значения, при которых необходимо очистить CurrentStateStorage.data
* Пример использования. Динамическое изменение фильтров на странице должно инициировать новую регистрацию существующих
* элементов на странице, но не должно инициировать смену страницы
*
*/
cleanUpDeps?: string[];
cleanUpDeps?: DependencyList;
};

const defaultDeps: string[] = [];
const defaultDeps: DependencyList = [];

/**
* Хелпер для отслеживания текущего состояния страницы, при навигации по приложению.
Expand Down
8 changes: 6 additions & 2 deletions packages/react/src/usePageAnalytics/usePanelPageAnalytics.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
import { CurrentStateStorage } from '@vkontakte/mini-apps-analytics';
import { useLayoutEffect } from 'react';
import { DependencyList, useLayoutEffect } from 'react';

export const usePanelPageAnalytics = (panelPageName: string, modalPageName: string | null | 0, deps: string[] = []) => {
export const usePanelPageAnalytics = (
panelPageName: string,
modalPageName: string | null | 0,
deps: DependencyList = []
) => {
/**
* Очищаем ранее собранную информацию по странице и инициируем новый сбор данных для страницы
*/
Expand Down

0 comments on commit 28f8dee

Please sign in to comment.