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.

Sign up for GitHub

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

Jump to bottom

Save ABI to localstorage on chain id 31337 #172

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Save ABI to localstorage on chain id 31337 #172

wants to merge 17 commits into from

Conversation

portdeveloper
Copy link
Member

This PR:

-> Saves abi on localhost
-> Gives the user option to change or remove the abi

Fixes #128

Known issues:

  • When the user loads an abi for the first time the user gets two success messages at the same time: one for saving the abi and another for loading it, which is not optimal.
msedge_Dqf025dKIr.mp4

@portdeveloper
Add save abi and load abi funcs to abi.ts
9276d74
@portdeveloper
Use abi from localstorage on homepage if the network is 31337
28602c2
@portdeveloper
Able to reload page on localhost
4d093b2
@portdeveloper
Add remove abi from local storage function to abi.ts
a1259f5
@portdeveloper
Able to remove abi from localstorage
03b1cca
@portdeveloper
Save abi for all contracts
b9a57e4
@portdeveloper
Add modal to remove the abi
1e667e9
@portdeveloper
Use better X to close the modal
8656bbc
@portdeveloper
Remove one eslint warning(exhaustive deps)
3bea975
@portdeveloper
Change wording in Configuration modal
e1b6adc
@portdeveloper
Reorder functions in abi.ts
496c763
@portdeveloper
Store all contracts under one local storage key
010f42c
@portdeveloper
Use type instead of interface for AbiStorage in abi.ts
95101ca
@portdeveloper
Save abi only for localhost + show the saved abi in Configuration modal
1c42b17
@portdeveloper
Remove double notifs for abi loaded
8f43162
@portdeveloper
Able to reflect changes in an instant
80edec1
@portdeveloper
Use slightly better notifications
6b8bfae
@vercel Vercel
Copy link

vercel bot commented Jan 21, 2025 •
edited
Loading

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 Jan 21, 2025 4:04pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

bug: the app does not allow reloading the page on localhost
1 participant
@portdeveloper