Skip to content
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

Adds export from useWatchBalance to hooks index file #840

Merged
merged 2 commits into from
May 9, 2024

Conversation

rin-st
Copy link
Member

@rin-st rin-st commented May 7, 2024

Description

Adds export from useWatchBalance to hooks index file. (I forgot to add when created it 🤷‍♂️ )
Plus sorted index file alphabetically

Additional Information

@rin-st rin-st changed the title Adds export from useWatchWalance to hooks index file Adds export from useWatchBalance to hooks index file May 7, 2024
@technophile-04
Copy link
Collaborator

Ohh nice catch ! Thanks @rin-st , also could you also add useTargetNetwork too to index.ts I think its a nice helpful hook and was also mentioned in #789 (comment)

Plus sorted index file alphabetically

Nicee! , I don't think our prettier is configured to do that so lol I think it again is going to get messed up in the future but yeah makes sense for now!

@rin-st
Copy link
Member Author

rin-st commented May 7, 2024

Ohh nice catch ! Thanks @rin-st , also could you also add useTargetNetwork too to index.ts I think its a nice helpful hook and was also mentioned in #789 (comment)

Thanks! Added

Nicee! , I don't think our prettier is configured to do that so lol I think it again is going to get messed up in the future but yeah makes sense for now!

Temporary order is better than nothing 😄 . We can add it later if needed, but not in this pr

Copy link
Collaborator

@Pabl0cks Pabl0cks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! If the refactor of all the imports needs to be done manually, I can help with that 🙌

Copy link
Member

@damianmarti damianmarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Thanks!

@technophile-04 technophile-04 merged commit 7b2642c into main May 9, 2024
1 check passed
@technophile-04 technophile-04 deleted the reexport-use-watch-balance branch May 9, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants