-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat: Show campaign total supply NFT #126
Conversation
WalkthroughThe recent updates involve adjustments to the Changes
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? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- packages/user/src/modules/campaign/campaign-detail/index.tsx (3 hunks)
Additional comments: 3
packages/user/src/modules/campaign/campaign-detail/index.tsx (3)
11-11: The import of
CampaignStatusEnum
is correctly placed and follows the existing import structure. Ensure that the imported enum is used consistently throughout the component.104-107: The conditional rendering logic using
CampaignStatusEnum.RUNNING
is appropriate and follows best practices for readability and maintainability.126-167: The new conditional block for rendering
nftCollection.broker
details is well-structured and follows the existing coding patterns. However, ensure that the hardcoded value99
in line 135 is intentional and consistent with the application's requirements.
Summary by CodeRabbit
Enhancements
CampaignDetail
component to enhance the display and conditions for rendering campaign-specific information.New Features
CampaignStatusEnum
.User Interface