Skip to content

readContracts using batchCall#65

Merged
WiseMrMusa merged 4 commits intomainfrom
develop
Mar 2, 2025
Merged

readContracts using batchCall#65
WiseMrMusa merged 4 commits intomainfrom
develop

Conversation

@WiseMrMusa
Copy link
Collaborator

This PR allows readContracts to make a batch single RPC call

@WiseMrMusa WiseMrMusa mentioned this pull request Mar 1, 2025
http,
} from 'starkweb';
import { sepolia } from 'starkweb/chains';
import { getStarknetId, getStarknetIdName } from 'starkweb/actions';

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import getStarknetIdName.

Copilot Autofix

AI 12 months ago

To fix the problem, we need to remove the unused import getStarknetIdName from the import statement on line 13. This will clean up the code and remove any confusion about unused imports.

  • Remove the unused import getStarknetIdName from the import statement.
  • Ensure that the remaining code functionality is not affected by this change.
Suggested changeset 1
app/starkweb-client/src/index.ts

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/app/starkweb-client/src/index.ts b/app/starkweb-client/src/index.ts
--- a/app/starkweb-client/src/index.ts
+++ b/app/starkweb-client/src/index.ts
@@ -12,3 +12,3 @@
 } from 'starkweb';
-import { getStarknetId, getStarknetIdName } from 'starkweb/actions';
+import { getStarknetId } from 'starkweb/actions';
 import { mainnet, sepolia } from 'starkweb/chains';
EOF
@@ -12,3 +12,3 @@
} from 'starkweb';
import { getStarknetId, getStarknetIdName } from 'starkweb/actions';
import { getStarknetId } from 'starkweb/actions';
import { mainnet, sepolia } from 'starkweb/chains';
Copilot is powered by AI and may make mistakes. Always verify output.
} from 'starkweb';
import { sepolia } from 'starkweb/chains';
import { getStarknetId, getStarknetIdName } from 'starkweb/actions';
import { mainnet, sepolia } from 'starkweb/chains';

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import sepolia.

Copilot Autofix

AI 12 months ago

To fix the problem, we need to remove the unused import sepolia from the import statement. This will clean up the code and remove any confusion about the usage of sepolia. The change should be made in the file app/starkweb-client/src/index.ts on line 14.

Suggested changeset 1
app/starkweb-client/src/index.ts

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/app/starkweb-client/src/index.ts b/app/starkweb-client/src/index.ts
--- a/app/starkweb-client/src/index.ts
+++ b/app/starkweb-client/src/index.ts
@@ -13,3 +13,3 @@
 import { getStarknetId, getStarknetIdName } from 'starkweb/actions';
-import { mainnet, sepolia } from 'starkweb/chains';
+import { mainnet } from 'starkweb/chains';
 
EOF
@@ -13,3 +13,3 @@
import { getStarknetId, getStarknetIdName } from 'starkweb/actions';
import { mainnet, sepolia } from 'starkweb/chains';
import { mainnet } from 'starkweb/chains';

Copilot is powered by AI and may make mistakes. Always verify output.
@WiseMrMusa WiseMrMusa merged commit efb8675 into main Mar 2, 2025
3 of 4 checks passed
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.

1 participant