coderabbitai[bot]
commented
1 month ago [!WARNING]
Rate limit exceeded
@martijnvdbrug has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 21 minutes and 27 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
Reviewing files that changed from the base of the PR and between b533f85f5233dd6351f4fd7eb822d7b481e3ddff and f36afc2c97107106bfd984833a82c947f2dc9b6f.
Walkthrough
The pull request updates the vendure-plugin-metrics package, introducing new features such as revenue per variant metrics, tax-inclusive calculations, and a maximum cache age for metrics. It modifies existing metrics to enhance data representation and updates the caching mechanism to include entry creation timestamps. Additionally, it refines the AverageOrderValueMetric and SalesPerProductMetric classes, while replacing the latter with the new RevenuePerProduct class. The changes also include updates to the component styles and test suite adjustments to reflect the new metric functionalities.
Changes
| File Path | Change Summary |
|---|---|
packages/vendure-plugin-metrics/CHANGELOG.md |
Updated to version 1.4.0; added revenue per variant metric, tax calculation options, sales-per-product formatting changes, and max cache age feature. |
packages/vendure-plugin-metrics/src/api/cache.ts |
Added EntryDate interface; updated Cache class to include maxAgeInSeconds for cache expiration and modified set and get methods for cache handling. |
packages/vendure-plugin-metrics/src/api/metrics/average-order-value.ts |
Simplified getTitle, changed pagination method, updated calculateDataPoints for clarity and tax inclusion. |
packages/vendure-plugin-metrics/src/api/metrics/revenue-per-product.ts |
Introduced RevenuePerProduct class for calculating revenue metrics by product variant. |
packages/vendure-plugin-metrics/src/api/metrics/sales-per-product.ts |
Changed metricType from Currency to Number; updated getTitle method. |
packages/vendure-plugin-metrics/src/index.ts |
Removed export for sales-per-product, added export for revenue-per-product. |
packages/vendure-plugin-metrics/src/metrics.plugin.ts |
Replaced SalesPerProductMetric with RevenuePerProduct in MetricsPlugin options. |
packages/vendure-plugin-metrics/src/ui/chartist/chartist.component.scss |
Updated $ct-area-opacity from 0.5 to 0.2. |
packages/vendure-plugin-metrics/src/ui/chartist/chartist.component.ts |
Added showArea property, updated event handling, modified animation parameters, and introduced a gradient for the chart. |
packages/vendure-plugin-metrics/src/ui/metrics-widget.ts |
Added imports for ViewChild and ElementRef, modified template styles, and updated loadChartData method logic. |
packages/vendure-plugin-metrics/test/metrics.spec.ts |
Enhanced test suite with new imports, updated assertions for metrics, and renamed variables for consistency. |
Possibly related PRs
- #526: Modifications to the
ChartistComponent, which is part of thevendure-plugin-metricspackage, indicating a direct connection in functionality and code structure.
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?
1.4.0 (2024-10-31)
Screenshots
Checklist
📌 Always:
👍 Most of the time:
📦 For publishable packages:
package.jsonCHANGELOG.mdSee this example