-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: analytics refactoring #13
Conversation
Are we need to delete extra analytics code from this repo ? |
038482b
to
ea26782
Compare
Preview is ready. |
48bef43
to
b4c2d31
Compare
We need it until we get 'ok' that new analytics in working properly |
@@ -70,9 +83,16 @@ export const Controls: React.FC<ControlsProps> = ({ | |||
}; | |||
|
|||
const handleTagSelect = (selectedTags: string[]) => { | |||
/** | |||
* @deprecated Metrika will be deleted after launch of analyticsEvents | |||
* https://st.yandex-team.ru/PAGECTR-7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think we don't need to write info about tickets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point
No description provided.