feat: Allow filtering for analytics page (frontend) #178
Conversation
|
@jyo142 is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
|
@nevo-david sorry for the very late PR! I have been pretty caught up with other life things :( |
| const query = | ||
| newState === 'none' ? `` : `?key=${param}&state=${newState}`; | ||
| router.replace(`/analytics${query}`); | ||
| const params = new URLSearchParams(); |
There was a problem hiding this comment.
changed to use URLSearchParams to make it easier to manage the dynamic params
jamesread
changed the title
jamesread
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| }, | ||
| [state, param] | ||
| [state, param, startDate, endDate] |
Check warning
Code scanning / ESLint
verifies the list of dependencies for Hooks like useEffect and similar Warning
| ); | ||
|
|
||
| const changeState = useCallback(() => { | ||
| changeStateUrl( | ||
| state === 'none' ? 'desc' : state === 'desc' ? 'asc' : 'none' | ||
| ); | ||
| }, [state, param]); | ||
| }, [state, param, startDate, endDate]); |
Check warning
Code scanning / ESLint
verifies the list of dependencies for Hooks like useEffect and similar Warning
| }, | ||
| [page, key, state] | ||
| [page, key, state, startDate, endDate] |
Check warning
Code scanning / ESLint
verifies the list of dependencies for Hooks like useEffect and similar Warning
| @@ -0,0 +1,129 @@ | |||
| import { FC, useCallback, useState } from 'react'; | |||
| import dayjs from 'dayjs'; | |||
| import { Calendar, RangeCalendar, TimeInput } from '@mantine/dates'; | |||
Check warning
Code scanning / ESLint
Disallow unused variables Warning
| @@ -0,0 +1,129 @@ | |||
| import { FC, useCallback, useState } from 'react'; | |||
| import dayjs from 'dayjs'; | |||
| import { Calendar, RangeCalendar, TimeInput } from '@mantine/dates'; | |||
Check warning
Code scanning / ESLint
Disallow unused variables Warning
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Allow filtering (frontend) for analytics page
Created a DateRangePicker component
The analytics component adds url parameters on change of the date range
Why was this change needed? (You can also link to an open issue here)
https://github.com/gitroomhq/gitroom/issues/147
Screen.Recording.2024-06-03.at.3.44.43.PM.mov