-
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
Clean-up v2 #27
Clean-up v2 #27
Conversation
Removes all things related to tokens other than tbtc-v2. This includes: - cleaning of useFetchTvl hook - removing transaction slice - cleaning of token slice - removing hooks related to nu, keep and t tokens - removing some other unrelated things
Removes unnecessary `useMulticall` call in `useFetchTvl` hook.
Remove modal queue from the store. We don't need it anymore.
The `focus-visible` lib was uninstalled in 5e73576
As of now, we do not use Storybook in this repository.
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.
One remark before it gets merged
Change all occurences of tbtcv2 to tbtc
`useMemo` hook is unnecessary now since creating `navItems` variable is not a costly operation anyway.
These icons were not used anymore.
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.
Approved
The `useContract` hook was removed in earlied commits.
TBTCV2 occurences were renamed to just TBTC.
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.
Approved
Work in progress...
Ref: #1
This continues the work from #12, #11, #10, #8 and #7. It removes all unnecessary things that was missed in those PRs.
TODO:
useFetchTvl
hooknode_modules