Closed ritch closed 3 months ago
[!WARNING]
Rate limit exceeded
@ritch has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 48 minutes and 47 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 381a400851b7302a7a65a70986610019cc84c159 and 167e011a4333c191652b500158f8890999b91653.
This update introduces a comprehensive analytics feature to the codebase, focusing on tracking user interactions and events. By leveraging Recoil state management and integrating Segment analytics, the system enables detailed event tracking and user behavior analysis across various application components. Key additions include React hooks for event tracking, state management enhancements, and utility scripts for configuring analytics, ESLint, Babel, and Vite.
File(s) | Change Summary |
---|---|
app/packages/analytics/.eslintrc |
Disables the rule react-hooks/rules-of-hooks . |
app/packages/analytics/.gitignore |
Adds exclusions for build artifacts, dependencies, and logs. |
app/packages/analytics/.prettierignore |
Adds the build directory to Prettier ignore list. |
app/packages/analytics/README.md |
Provides documentation for FiftyOne Analytics package. |
app/packages/analytics/babel.config.js |
Configures Babel presets for Node.js and TypeScript. |
app/packages/analytics/vite.config.ts |
Configures Vite for ES module format and ES2015 target. |
app/packages/analytics/src/index.ts |
Exports modules for event tracking and analytics state management. |
app/packages/analytics/src/keys.ts |
Exports DEFAULT_WRITE_KEYS for "dev" and "prod" environments. |
app/packages/analytics/src/state.ts |
Introduces Recoil atom analyticsInfo for holding AnalyticsInfo . |
app/packages/analytics/src/useAnalyticsInfo.ts |
Introduces useAnalyticsInfo hook for managing AnalyticsInfo state. |
app/packages/analytics/src/useTrackEvent.ts |
Adds useTrackEvent hook for tracking events. |
app/packages/analytics/src/usingAnalytics.ts |
Adds functionality for managing analytics via Analytics class. |
app/packages/app/src/components/Nav.tsx |
Modifies useGA and introduces hooks from recoil . |
app/packages/app/src/pages/datasets/DatasetPage.tsx |
Includes <EventTracker /> component. |
app/packages/components/src/components/ErrorBoundary/ErrorBoundary.tsx |
Adds TrackFallback function for enhanced error tracking using useEffect . |
app/packages/core/src/components/Actions/ActionsRow.tsx |
Introduces useTrackEvent to track "open_color_settings" events. |
app/packages/core/src/components/Actions/DynamicGroup.tsx |
Adds useTrackEvent to track dynamic group actions. |
app/packages/core/src/components/Actions/Tagger.tsx |
Uses useTrackEvent to track events in submitWrapper function. |
app/packages/core/src/components/EventTracker.tsx |
Adds components for tracking view state changes using Recoil. |
app/packages/embeddings/src/Embeddings.tsx |
Adds a line to throw "hello world" error inside useEffect hook. |
app/packages/operators/src/built-in-operators.ts |
Adds TrackEvent class defining a new operator for event tracking. |
app/packages/operators/src/operators.ts |
Adds trackOperatorExecution and integrates it in executeOperatorWithContext for event logging. |
app/packages/operators/src/state.ts |
Incorporates useAnalyticsInfo() in useExecutionContext function. |
app/packages/spaces/src/components/AddPanelItem.tsx |
Utilizes useTrackEvent to track panel item clicks. |
app/packages/state/src/recoil/index.ts |
Adds export for info module. |
app/packages/state/src/recoil/info.ts |
Introduces info atom. |
fiftyone/operators/operations.py |
Adds track_event method for event tracking in the App. |
sequenceDiagram
participant ReactComponent as React Component
participant Hook as useTrackEvent
participant Recoil as Recoil State
participant Analytics as Analytics Service
ReactComponent->>Hook: Call useTrackEvent()
Hook->>Recoil: Access analyticsInfo atom
ReactComponent->>Hook: Track Event
Hook->>Analytics: Send Event Data
Analytics-->>Hook: Event Tracking Confirmation
Hook-->>ReactComponent: Event Tracking Complete
In Recoil's state, the atoms play, Tracking events day by day. With hooks and calls, the analytics bloom, In every component, there's now room. Errors tracked, views all seen, Our codebase shines with a data sheen. 🌟🐇
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?
Note: event tracking is disabled by default. Only environments that inject
window.analytics
will make use of these new event tracking features.When enabled all operator executions and all view changes will be tracked in the associated event aggregator.
Adds a new
ctx.ops.track_event(event_name, properties)
operator for manually tracking events in operators/panels.Summary by CodeRabbit
New Features
ErrorBoundary
component.Bug Fixes
Documentation