Closed kemuru closed 3 months ago
The recent updates to the Kleros SDK improve flexibility and security by allowing the direct passing of the Alchemy API key in various functions, reducing reliance on environment variables. Enhancements in type safety across multiple files and a streamlining of the testing process contribute to a clearer and more efficient development experience for users.
File Path | Change Summary |
---|---|
kleros-sdk/README.md | Clarified instructions for data mapping tests, simplified test command to yarn test , and retained guidance on environment variables. |
kleros-sdk/package.json | Removed dotenv from devDependencies , indicating a change in environment variable management. |
kleros-sdk/src/dataMappings/actions/callAction.ts | Modified callAction to accept alchemyApiKey as a parameter, enhancing function flexibility. |
kleros-sdk/src/dataMappings/actions/eventAction.ts | Updated eventAction to include alchemyApiKey as a parameter. |
kleros-sdk/src/dataMappings/executeActions.ts | Changed context parameter type to Record<string, unknown> , passing context.alchemyApiKey to actions. |
kleros-sdk/src/dataMappings/utils/replacePlaceholdersWithValues.ts | Updated context parameter type to Record<string, unknown> . |
kleros-sdk/test/dataMappings.test.ts | Updated tests for new API interactions, including changes to the GraphQL endpoint and ABI definitions. |
sequenceDiagram
participant User
participant SDK
participant API
User->>SDK: Call Function with alchemyApiKey
SDK->>API: Pass alchemyApiKey to API request
API-->>SDK: Return Data
SDK-->>User: Return Result
🐰 In the meadow so bright,
A rabbit hops with delight,
With keys of Alchemy near,
Now coding brings us cheer!
No more worries, no more fuss,
Just run tests and trust in us! 🌿✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Name | Link |
---|---|
Latest commit | 4c17ea8dd523574d0311a43fddb2f2e9cfd95ad1 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-neo/deploys/66ad200204a36a0008c1db4d |
Deploy Preview | https://deploy-preview-1663--kleros-v2-neo.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Name | Link |
---|---|
Latest commit | 4c17ea8dd523574d0311a43fddb2f2e9cfd95ad1 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-university/deploys/66ad20027e4d2400083617fe |
Deploy Preview | https://deploy-preview-1663--kleros-v2-university.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Name | Link |
---|---|
Latest commit | 4c17ea8dd523574d0311a43fddb2f2e9cfd95ad1 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-testnet/deploys/66ad2002199cd20008503180 |
Deploy Preview | https://deploy-preview-1663--kleros-v2-testnet.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Code Climate has analyzed commit 4c17ea8d and detected 10 issues on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Complexity | 1 |
Duplication | 8 |
Style | 1 |
View more on Code Climate.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
needs NPM package redeployment., but shoud wait until 1663 is merged too
PR-Codex overview
The focus of this PR is to update API key handling in various files and improve testing setups.
Detailed summary
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Refactor
Chores