Skip to content

Fixed the issue where issuer descriptions were not displayed#480

Open
iamlokanath wants to merge 1 commit intoinji:developfrom
iamlokanath:injiweb-1348
Open

Fixed the issue where issuer descriptions were not displayed#480
iamlokanath wants to merge 1 commit intoinji:developfrom
iamlokanath:injiweb-1348

Conversation

@iamlokanath
Copy link

@iamlokanath iamlokanath commented Jan 8, 2026

The description now appears below the issuer title in the issuer cards
It only renders when a description is available (since it's optional)
Styled to differentiate from the title while maintaining visual consistency

Summary by CodeRabbit

  • New Features
    • Enhanced item boxes with conditional description display, allowing additional contextual information to be shown when provided.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 8, 2026

Walkthrough

The ItemBox component has been updated to support conditional rendering of an optional description field. When a description property is provided, it displays beneath the title with specific styling and a test identifier.

Changes

Cohort / File(s) Summary
UI Component Enhancement
inji-web/src/components/Common/ItemBox.tsx
Added conditional rendering of optional description paragraph with data-testid attribute; minor whitespace adjustment in import formatting

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A box that now speaks its mind,
With descriptions of every kind,
When present, it shows with pride,
Where the title sits side by side,
ItemBox is dressed up fine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding conditional rendering of issuer descriptions in the ItemBox component that were previously not displayed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e4c7129 and 010c244.

📒 Files selected for processing (1)
  • inji-web/src/components/Common/ItemBox.tsx
🔇 Additional comments (2)
inji-web/src/components/Common/ItemBox.tsx (2)

28-35: Implementation looks good!

The conditional rendering is correctly implemented with appropriate styling that maintains visual hierarchy (smaller font, lighter color) while ensuring consistency with the title element (same width, centered, word-breaking). The addition of data-testid supports test coverage.

Minor notes for awareness:

  • Ensure issuer descriptions are sanitized if they come from external sources (likely not an issue if admin-controlled).
  • The text-gray-600 color should meet WCAG contrast requirements on white background for normal text.

28-35: No changes needed. The description property is correctly typed as optional (description?: string) in ItemBoxProps, and the conditional rendering implementation properly handles the optional value.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Lkanath Panda <lokanthpanda128@gmai.com>
@iamlokanath
Copy link
Author

INJIWEB-1348.mp4

@@ -1,5 +1,5 @@
import React from 'react';
import {ItemBoxProps} from '../../types/components';
import { ItemBoxProps } from '../../types/components';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let the whitespace changes be avoided

@Rudhhi-Shah-14
Copy link
Contributor

Also do update the ItemBox test file accordingly.

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.

2 participants