Daily Stats#88
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| import DailyStatsChart from './DailyStatsChart' | ||
| import FetchMessage from './FetchMessage' | ||
|
|
||
| const DAILY_STATS_URL = `${process.env.API_BASE_URL}/api/secure/dailystats?fromDate=2025-01-01` |
There was a problem hiding this comment.
| const DAILY_STATS_URL = `${process.env.API_BASE_URL}/api/secure/dailystats?fromDate=2025-01-01` | |
| const DAILY_STATS_URL = `${process.env.API_BASE_URL}/api/secure/dailystats?fromDate=${someGeneratedData}` |
Not a formal review but this jumped out to me! Overall this looks exciting thanks for your work
amy-corson-ibigroup
left a comment
amy-corson-ibigroup
left a comment
There was a problem hiding this comment.
Looking really good! Just a few nitpicks and questions
| }, [auth0, currentlyDownloadingFile]) | ||
|
|
||
| let files = Object.keys(swrData).length > 0 ? swrData?.data?.data : [] | ||
| let files = Object.keys(swrData).length > 0 ? swrData?.data?.data || [] : [] |
There was a problem hiding this comment.
Could this just be let files = swrData?.data?.data || []
There was a problem hiding this comment.
There are a few functions getting flagged for missing return types here
There was a problem hiding this comment.
Same thing here, a few missing return types
|
|
||
| render() { | ||
| const { entityType, records, series } = this.props | ||
| if (records.length === 0) return null |
There was a problem hiding this comment.
Could this just be if (!records.length)?
|
|
||
| render() { | ||
| const { data, entityType, title } = this.props | ||
| if (data.length === 0) return null |
There was a problem hiding this comment.
Same here, could it just be if (!data.length)?
4 participants
This PR adds a tab page with charts for daily number of trip requests, and OTP users with trip requests, and total number of OTP users. Daily stats are populated only for OTP-middleware deployments that include ibi-group/otp-middleware#349.