Closed Omri-Levy closed 3 weeks ago
Latest commit: 576a52d199b2938bcc96c167338189160616999f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
[!WARNING]
Rate limit exceeded
@Omri-Levy has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 54 minutes and 28 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 93040c11153b3f99ff1a3368dc4fa36c39c53f10 and 576a52d199b2938bcc96c167338189160616999f.
The changes primarily enhance the Select_
, RiskSelect
, and SwitchesList
components within a merchant monitoring feature. This involves adding new properties like placeholder
and disabled
to the Select_
component, renaming and updating the RiskSelect
component, and including a defaultChecked
property in SwitchesList
. Additionally, it modifies the configuration logic for checks within the useMerchantMonitoringCreateBusinessReportPageLogic
file, removing redundant configurations.
Files & Paths | Change Summary |
---|---|
apps/.../Select_/Select_.tsx , apps/.../Select_/interfaces.ts |
Added placeholder and disabled props to Select_ component and its interface. |
apps/.../MerchantMonitoringCreateCheck/MerchantMonitoringCreateCheck.page.tsx |
Renamed RiskDropdown to RiskSelect and updated its usage accordingly. |
apps/.../components/RiskSelect/RiskSelect.tsx |
New file introducing RiskSelect component for selecting risk levels using react-hook-form and Select_ . |
apps/.../components/SwitchesList/SwitchesList.tsx |
Added defaultChecked property to options array in SwitchesList component. |
apps/.../hooks/.../useMerchantMonitoringCreateBusinessReportPageLogic.tsx |
Updated checksConfiguration with new properties like defaultChecked and disabled ; removed disabledChecksConfiguration . |
🐰 Within the lines of code, we've sown,
Choices bloom where once unknown.
Switches check and select with ease,
Logic flows through fields of these.
Risk assessed with dropdown flair,
Changes made with utmost care. 🌟
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?
Summary by CodeRabbit
New Features
Select_
component withplaceholder
anddisabled
props for improved dropdown functionality.RiskSelect
component for better risk level selection, utilizingreact-hook-form
for seamless form management.defaultChecked
property toSwitchesList
component for managing default switch states.Refactor
RiskDropdown
toRiskSelect
for consistency in risk-related configurations.disabledChecksConfiguration
and integrating new properties directly intochecksConfiguration
.