Closed Harman-singh-waraich closed 1 month ago
The changes primarily revolve around updating the structure and usage of aliases within the dispute resolution components. This includes altering type definitions, interface declarations, and logic to support a new format for aliases (with name
and address
properties). Refactoring ensures consistent handling of aliases across different parts of the application.
Files | Change Summaries |
---|---|
web/src/components/DisputePreview/Alias.tsx |
IAlias interface updated to include name: string and address: '0x${string}' properties; AliasDisplay component now receives name and address instead of alias , adjusted logic to use these properties. |
web/src/components/DisputePreview/DisputeContext.tsx |
Changed iteration logic to map over keys of disputeDetails.aliases and pass name and address props to AliasDisplay components. |
web/src/context/NewDisputeContext.tsx |
Restructured Alias type to AliasArray , modified IDisputeTemplate interface to include aliases property, and adjusted constructDisputeTemplate logic to handle aliasesArray . |
web/src/pages/Resolver/NavigationButtons/NextButton.tsx |
Updated areFilledAddressesValid logic to check for name field in addition to address and isValid . |
web/src/pages/Resolver/Parameters/NotablePersons/PersonFields.tsx , index.tsx |
Renamed Alias type to AliasArray ; updated references to aliases to aliasesArray in logic for handling aliases and maintaining consistency in data structure. |
web/src/pages/Resolver/index.tsx |
Restructured JSX elements and adjusted conditional rendering logic within DisputeResolver component to handle scenarios where the user is connected or disconnected, impacting the display of various components within the resolver page. |
sequenceDiagram
participant User
participant DisputeResolver
participant AliasDisplay
participant Object.keys
User->>DisputeResolver: Open Dispute Page
DisputeResolver->>Object.keys: Get alias keys
Object.keys-->>DisputeResolver: Return alias keys
DisputeResolver->>AliasDisplay: Render with name and address
AliasDisplay->>User: Display name and resolved address
In the fields of code we roam,
Aliases find their proper home.
Names and addresses, side by side,
Ensuring nothing's left to hide.
As we refactor, lines align,
A clearer path, our code does shine.
🎉 Here's to progress, inch by inch! 🎉
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 | 2abad9ff3f7609416b834a69da2c7ec1d0526e55 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2/deploys/667c3463cf350500083444cd |
Deploy Preview | https://deploy-preview-1629--kleros-v2.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 | 2abad9ff3f7609416b834a69da2c7ec1d0526e55 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-university/deploys/667c34637552b90008fc9263 |
Deploy Preview | https://deploy-preview-1629--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 | 2abad9ff3f7609416b834a69da2c7ec1d0526e55 |
Latest deploy log | https://app.netlify.com/sites/kleros-v2-neo/deploys/667c346320e7150008466c2b |
Deploy Preview | https://deploy-preview-1629--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.
Code Climate has analyzed commit 2abad9ff and detected 4 issues on this pull request.
Here's the issue category breakdown:
Category | Count |
---|---|
Style | 4 |
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
PR-Codex overview
The focus of this PR is to refactor how aliases are handled in the codebase, switching from arrays to objects for better data management.
Detailed summary
Summary by CodeRabbit
New Features
AliasDisplay
component to usename
andaddress
properties for fetching and displaying address information.DisputeContext
component to iterate overdisputeDetails.aliases
keys for displaying alias information.Refactor
Alias
type toAliasArray
and updated references in various components to ensure consistency.NextButton
component to include a check for thename
field when validating addresses.