Fixed ui for organization details page

[sujitaw]

What

• updated badge placement position for primary did
• fixed the complete page load when setting primary did
• showed org did complete without ellipsis
• added acknowledgement on copying data

---

Important

Improved UI for organization details page with better badge placement, full DID display, copy acknowledgment, and optimized DID setting.

• UI Enhancements:
  • Updated badge placement for primary DID in DidListComponent.tsx.
  • Displayed full org DID without ellipsis in OrganizationDetails.tsx.
  • Added acknowledgment icon when copying data in OrganizationDetails.tsx and DidListComponent.tsx.
• Performance:
  • Replaced full page reload with data fetch in setPrimaryDid() in DidListComponent.tsx.
• Misc:
  • Added DateTooltip component to display formatted date in DidListComponent.tsx.
  • Minor refactoring and state management improvements in OrganizationDetails.tsx.

^{This description was created by for e735473. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to 599de44 in 2 minutes and 47 seconds. Click for details.

• Reviewed 267 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 4 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. nextjs/src/features/wallet/DidListComponent.tsx:645

• Draft comment:
  Verify that the use of '!important' in the Tailwind class 'max-w-2xl!' is intentional and supported by your styling setup.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

2. nextjs/src/features/wallet/DidListComponent.tsx:187

• Draft comment:
  Casting the response to a string for error messaging (setErrorMsg(response as string)) may not yield a meaningful message. Consider extracting a proper error message from the response.
• Reason this comment was not posted:
  Comment was on unchanged code.

3. nextjs/src/features/wallet/DidListComponent.tsx:432

• Draft comment:
  In generatePolygonKeyValuePair, consider using the freshly generated private key directly instead of 'privateKeyValue || privateKey' to ensure consistency.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

4. nextjs/src/features/wallet/DidListComponent.tsx:457

• Draft comment:
  Both this file and OrganizationDetails define a CopyDid component with similar functionality. Consider extracting a shared copy-to-clipboard component to reduce duplication.
• Reason this comment was not posted:
  Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_S5kWf7q0k63eMKSQ

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[pranalidhanavade]

LG

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed e735473 in 37 seconds. Click for details.

• Reviewed 12 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. nextjs/src/features/wallet/DidListComponent.tsx:181

• Draft comment:
  Good update: removing the commented-out window.location.reload() prevents a full page reload. Using getData() to update the state is more efficient.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_dtrdttOdpKXFFr6N

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud