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

Merge stores into one, fix chain being different in store #164

Merged
merged 4 commits into from
Dec 3, 2024

Conversation

portdeveloper
Copy link
Member

Description

This PR

-> Removes useTargetNetwork hook since we are already able to get the network from the store, and the chain that matters in abi.ninja is not the wallet's chain but the selected chain in the NetworksDropdown
-> Updates network in the store as soon as a network is selected in the NetworksDropdown menu
-> Merges abi ninja store into global store since it is cleaner that way.

Testing

I have tested it locally with the existing cypress tests and passes 5/5 on cypress tests

_Closes #158

Copy link

vercel bot commented Nov 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
abi-ninja-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 3, 2024 7:19am

Copy link
Member

@technophile-04 technophile-04 left a comment

Choose a reason for hiding this comment

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

LOVE IT! Tysm Port! Lol this has been eating me for a while and you updated it nicely all changes look good!

Did some basic testing and everything works nicely, lets wait for Pablo to do his rigorous test 🙌

Copy link
Member

@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.

GJ Port!! 🙏

I've tested the same use cases than last week, and is working nicely! 🔥

PD: I can retest if needed if you push any change that could break stuff 👌

@technophile-04
Copy link
Member

Ok I updated the actions test file inline with #163 API and now everything seems to be passing, merging this 🙌

Thanks @portdeveloper and @Pabl0cks for testing!

@technophile-04 technophile-04 merged commit 941c8f5 into main Dec 3, 2024
4 checks passed
@portdeveloper portdeveloper deleted the refactor-store branch December 4, 2024 17:34
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.

3 participants