-
Notifications
You must be signed in to change notification settings - Fork 30
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
Sweep: Allow users to specify pre-deployed addresses in the contract deployment tab and the contract interaction tab. #188
Comments
Here's the PR! #189.⚡ Sweep Basic Tier: I'm using GPT-4. You have 4 GPT-4 tickets left for the month and 2 for the day.
For more GPT-4 tickets, visit our payment portal. For a one week free trial, try Sweep Pro (unlimited GPT-4 tickets). Actions (click)
Install Sweep Configs: Pull Request
Step 1: 🔎 SearchingI found the following snippets in your repository. I will now analyze these snippets and come up with a plan. Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.starknet-remix-plugin/README.md Lines 65 to 164 in 62db8aa
starknet-remix-plugin/api/scripts/deploy.js Lines 1 to 55 in 62db8aa
starknet-remix-plugin/plugin/src/utils/utils.ts Lines 1 to 122 in 62db8aa
starknet-remix-plugin/plugin/src/features/Deployment/index.tsx Lines 54 to 211 in 62db8aa
Step 2: ⌨️ Coding
Modify plugin/src/features/Deployment/index.tsx with contents:
Create plugin/src/utils/fetchContract.ts with contents:
Modify plugin/src/features/Deployment/index.tsx with contents:
Modify plugin/src/features/Deployment/index.tsx with contents: The syntax error in the file needs to be corrected. The misplaced parenthesis at line 116 should be moved to line 113, after the comment `// handle class data`. This will correctly close the try block that starts at line 108. Ensure that the indentation of the code is correct after moving the parenthesis.
Step 3: 🔁 Code ReviewI have finished reviewing the code for completeness. I did not find errors for 🎉 Latest improvements to Sweep:
💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. |
The user can select a drop-down of compiled classes to deploy. They should also be able to pass in a deployed class' address to deploy a contract of that type. (Note: contracts are deployed using a 'declare' and then a 'deploy' step.). Similarly, for deployed contracts in the interact tab, you should be able to connect to a pre-deployed contract.
For the UI, the tab itself should have a little plus symbol on it whose tooltip should read "connect". It should pop up a modern input box over all the content at the bottom of the page with some example text of "0xClassHash" for a class and "0xContractAddress" for a contract. The tab should then open and the redeployed asset available in the selection drop-down.
The following RPC endpoints will be of use.
Getting a declared class according to its hash:
The text was updated successfully, but these errors were encountered: