Closed greinard closed 1 month ago
[!WARNING]
Rate limit exceeded
@greinard has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 13 minutes and 45 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 d0a5af1eb298b6f84a863190221e2b9b27340c81 and 633fe8d38a2c2effb6806ef47b82b5b6ffd1c877.
The recent changes enhance the ConnectDexcom integration within the application, introducing new components, improved localization support, and modifications to existing files for seamless Dexcom account connectivity. The architecture has been refined for better usability, ensuring that Dexcom devices are effectively supported with enhanced functionality to prevent errors during preview states and updated dependencies.
Files | Change Summary |
---|---|
package.json |
Updated version to 2.26.1-ConnectDexcom.3 and dependency for @careevolution/mydatahelps-js from ^3.15.0 to ^3.23.0 . |
src/components/container/ConnectDevice/... |
Enhanced logic in ConnectDevice to prevent connection attempts when in preview state. |
src/components/container/ConnectDevicesMenu/... |
Added Dexcom provider, corrected Omron name, and introduced dexcomEnabled setting for account management. |
src/components/container/ConnectDevicesMenu/ConnectDevicesMenu.stories.tsx |
Updated accountTypes to include "Dexcom". |
src/components/container/ConnectDevicesMenu/ConnectDevicesMenu.tsx |
Updated DeviceAccountType to include "Dexcom", modified filtering logic based on user settings, and enhanced handling for AppleHealth menu items. |
src/components/container/ConnectDexcom/... |
Introduced ConnectDexcom component for user connections and a Storybook story for visualization. |
src/helpers/providerIDs.ts |
Refactored provider ID functions and added getDexcomProviderID . |
src/helpers/strings-*.ts (de, en, es, fr, it, nl, pl, pt) |
Added localization strings for Dexcom integration in respective language files. |
src/components/container/index.ts |
Exported ConnectDexcom and ConnectDexcomPreviewState for improved module organization. |
sequenceDiagram
participant User
participant ConnectDevice
participant ConnectDexcom
User->>ConnectDevice: Initiate connection
ConnectDevice->>ConnectDevice: Check previewState
alt If previewState is active
ConnectDevice-->>User: Abort connection
else
ConnectDevice->>ConnectDexcom: Proceed to connect Dexcom
ConnectDexcom->>User: Request Dexcom credentials
User-->>ConnectDexcom: Provide credentials
ConnectDexcom-->>User: Confirm connection
end
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?
Overview
This branch adds a new
ConnectDexcom
component for prompting the participant to connect to their Dexcom account. The implementation mirrors those of Fitbit and Garmin.It also adds support for Dexcom to the existing
ConnectDevicesMenu
component.We do not yet have the Dexcom production provider ID. I will update the helper function as soon as we have that. For now, it returns the development provider ID for both.
Security
REMINDER: All file contents are public.
Describe briefly what security risks you considered, why they don't apply, or how they've been mitigated.
Checklist
Testing
Documentation
Summary by CodeRabbit
New Features
Bug Fixes
Chores