-
Notifications
You must be signed in to change notification settings - Fork 18
Closed
Description
Discussed in #155
Originally posted by milooy August 28, 2021
첫 논의 스레드: https://www.notion.so/RAP-98df273e1c8b49d9be89107423a9425d#db3e8d475fc24d29a369b6e723a37e61
e.g.
const referrer = useQueryParam('referrer');
const [userNo, setUserNo] = useState('');
useAnalyticsPageView({referrer, userNo})
위 상황에서 referrer나 userNo가 바뀌면 PageView가 또 찍힙니다.
[대안]
params를 useEffect depths에서 제거
React.useEffect(() => {
analytics.onPageView(params);
}, [analytics]);
문제: useQueryParam등 꼭 필요한 인자가 비동기로 늦게 들어오면? (e.g. referrer: '' -> referrer: 'facebook')
useAnalyticsPageView에 함수도 넘길 수 있도록 해주면 어떠려나요
useAnalyticsPageView(() => {
if (referrer === '') {
return;
}
return {referrer}
})
의견 부탁스
greatSuminigreatSumini
Metadata
Metadata
Assignees
Labels
No labels