Closed Harman-singh-waraich closed 1 month ago
The changes involve modifications to the usePopulatedDisputeData
and useIsCrossChainDispute
hooks. The usePopulatedDisputeData
hook now includes an error
property in its return value for improved error handling, along with a logging statement for debugging. The useIsCrossChainDispute
hook has been updated to log its internal state and throws a generic error instead of a specific message. The overall structure of both hooks remains unchanged.
File Path | Change Summary |
---|---|
web/src/hooks/queries/usePopulatedDisputeData.ts | Modified to include an error property in the return value and added a console.log statement for debugging. |
web/src/hooks/queries/useIsCrossChainDispute.ts | Updated to log internal state values and throw a generic Error instead of a specific error message. |
In the code where disputes lay,
Errors now find a clearer way.
With logs to guide and help us see,
Cross-chain paths as smooth as can be.
A hop, a skip, through data we glide,
In the world of hooks, we take great pride! 🐰✨
Name | Link |
---|---|
Latest commit | b7cf1de446a065dacfe31263e5e7a907b413ade6 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-testnet-devtools/deploys/6710ce96d6f0640008b447f8 |
Deploy Preview | https://deploy-preview-1719--kleros-v2-testnet-devtools.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 | b7cf1de446a065dacfe31263e5e7a907b413ade6 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-testnet/deploys/6710ce96de243300082e5e1c |
Deploy Preview | https://deploy-preview-1719--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.
Name | Link |
---|---|
Latest commit | b7cf1de446a065dacfe31263e5e7a907b413ade6 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-university/deploys/6710ce9669481e00086efc2b |
Code Climate has analyzed commit b7cf1de4 and detected 28 issues on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Complexity | 3 |
Duplication | 6 |
Style | 19 |
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
Name | Link |
---|---|
Latest commit | b7cf1de446a065dacfe31263e5e7a907b413ade6 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-neo/deploys/6710ce9646cd4d00088dfa1f |
Deploy Preview | https://deploy-preview-1719--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.
⚠️ testing
PR-Codex overview
This PR focuses on enhancing debugging and error tracking in the
useIsCrossChainDispute
andusePopulatedDisputeData
hooks by adding console logging for key variables.Detailed summary
console.log
statement inuseIsCrossChainDispute
to logisEnabled
,publicClient
,arbitrableAddress
, anddisputeID
.usePopulatedDisputeData
to includeerror
in the destructured return fromuseIsCrossChainDispute
.console.log
statement inusePopulatedDisputeData
to logcrossChainData
,error
,isError
,externalDisputeID
,disputeID
, andarbitrableAddress
.Summary by CodeRabbit
New Features
usePopulatedDisputeData
hook.Bug Fixes
useIsCrossChainDispute
hook to throw a generic error.