Fixed paging in events/sessions.

This commit is contained in:
Mike Cao 2025-07-13 21:10:59 -07:00
parent 8c26e310f7
commit 465a03f987
9 changed files with 26 additions and 19 deletions

View file

@ -23,7 +23,7 @@ export function WebsiteTabs() {
}, },
{ {
id: 'sessions', id: 'sessions',
label: formatMessage(labels.users), label: formatMessage(labels.sessions),
icon: <User />, icon: <User />,
path: '/sessions', path: '/sessions',
}, },

View file

@ -13,7 +13,7 @@ export function EventsDataTable({
children?: ReactNode; children?: ReactNode;
}) { }) {
const { formatMessage, labels } = useMessages(); const { formatMessage, labels } = useMessages();
const queryResult = useWebsiteEventsQuery(websiteId); const query = useWebsiteEventsQuery(websiteId);
const [view, setView] = useState('all'); const [view, setView] = useState('all');
const buttons = [ const buttons = [
@ -22,12 +22,12 @@ export function EventsDataTable({
label: formatMessage(labels.all), label: formatMessage(labels.all),
}, },
{ {
id: 'page', id: 'views',
label: formatMessage(labels.page), label: formatMessage(labels.views),
}, },
{ {
id: 'event', id: 'events',
label: formatMessage(labels.event), label: formatMessage(labels.events),
}, },
]; ];
@ -37,7 +37,7 @@ export function EventsDataTable({
return ( return (
<DataGrid <DataGrid
query={queryResult} query={query}
allowSearch={true} allowSearch={true}
autoFocus={false} autoFocus={false}
allowPaging={true} allowPaging={true}

View file

@ -2,7 +2,7 @@ import { z } from 'zod';
import { getQueryFilters, parseRequest } from '@/lib/request'; import { getQueryFilters, parseRequest } from '@/lib/request';
import { unauthorized, json } from '@/lib/response'; import { unauthorized, json } from '@/lib/response';
import { canViewWebsite } from '@/lib/auth'; import { canViewWebsite } from '@/lib/auth';
import { dateRangeParams, pagingParams, filterParams } from '@/lib/schema'; import { dateRangeParams, pagingParams, filterParams, searchParams } from '@/lib/schema';
import { getWebsiteEvents } from '@/queries'; import { getWebsiteEvents } from '@/queries';
export async function GET( export async function GET(
@ -13,6 +13,7 @@ export async function GET(
...dateRangeParams, ...dateRangeParams,
...filterParams, ...filterParams,
...pagingParams, ...pagingParams,
...searchParams,
}); });
const { auth, query, error } = await parseRequest(request, schema); const { auth, query, error } = await parseRequest(request, schema);
@ -27,7 +28,7 @@ export async function GET(
return unauthorized(); return unauthorized();
} }
const filters = await getQueryFilters(query); const filters = await getQueryFilters(query, websiteId);
const data = await getWebsiteEvents(websiteId, filters); const data = await getWebsiteEvents(websiteId, filters);

View file

@ -1,5 +1,5 @@
import { z } from 'zod'; import { z } from 'zod';
import { parseRequest, getQueryFilters, setWebsiteDate } from '@/lib/request'; import { parseRequest, getQueryFilters } from '@/lib/request';
import { unauthorized, json } from '@/lib/response'; import { unauthorized, json } from '@/lib/response';
import { canViewWebsite } from '@/lib/auth'; import { canViewWebsite } from '@/lib/auth';
import { filterParams, timezoneParam, unitParam } from '@/lib/schema'; import { filterParams, timezoneParam, unitParam } from '@/lib/schema';
@ -29,7 +29,7 @@ export async function GET(
return unauthorized(); return unauthorized();
} }
const filters = await setWebsiteDate(websiteId, await getQueryFilters(query)); const filters = await getQueryFilters(query, websiteId);
const data = await getEventMetrics(websiteId, filters); const data = await getEventMetrics(websiteId, filters);

View file

@ -11,7 +11,8 @@ export function useWebsiteEventsQuery(websiteId: string, options?: ReactQueryOpt
return usePagedQuery({ return usePagedQuery({
queryKey: ['websites:events', { websiteId, ...date, ...filters }], queryKey: ['websites:events', { websiteId, ...date, ...filters }],
queryFn: () => get(`/websites/${websiteId}/events`, { ...date, ...filters, pageSize: 20 }), queryFn: pageParams =>
get(`/websites/${websiteId}/events`, { ...date, ...filters, ...pageParams, pageSize: 20 }),
enabled: !!websiteId, enabled: !!websiteId,
...options, ...options,
}); });

View file

@ -15,11 +15,12 @@ export function useWebsiteSessionsQuery(
return usePagedQuery({ return usePagedQuery({
queryKey: ['sessions', { websiteId, modified, ...params, ...date, ...filters }], queryKey: ['sessions', { websiteId, modified, ...params, ...date, ...filters }],
queryFn: () => { queryFn: pageParams => {
return get(`/websites/${websiteId}/sessions`, { return get(`/websites/${websiteId}/sessions`, {
...params,
...filters,
...date, ...date,
...filters,
...pageParams,
...params,
pageSize: 20, pageSize: 20,
}); });
}, },

View file

@ -103,9 +103,13 @@ export async function getQueryFilters(
params: Record<string, any>, params: Record<string, any>,
websiteId?: string, websiteId?: string,
): Promise<QueryFilters> { ): Promise<QueryFilters> {
const dateRange = await setWebsiteDate(websiteId, getRequestDateRange(params)); const dateRange = getRequestDateRange(params);
const filters = getRequestFilters(params); const filters = getRequestFilters(params);
const segments = await getRequestSegments(websiteId, params); const segments = websiteId ? await getRequestSegments(websiteId, params) : null;
if (websiteId) {
await setWebsiteDate(websiteId, dateRange);
}
return { return {
...dateRange, ...dateRange,

View file

@ -50,7 +50,6 @@ export const pagingParams = {
export const sortingParams = { export const sortingParams = {
orderBy: z.string().optional(), orderBy: z.string().optional(),
sortDescending: z.string().optional(),
}; };
export const userRoleParam = z.enum(['admin', 'user', 'view-only']); export const userRoleParam = z.enum(['admin', 'user', 'view-only']);

View file

@ -16,6 +16,7 @@ async function relationalQuery(websiteId: string, filters: QueryFilters) {
const { filterQuery, dateQuery, cohortQuery, queryParams } = parseFilters({ const { filterQuery, dateQuery, cohortQuery, queryParams } = parseFilters({
...filters, ...filters,
websiteId, websiteId,
search: `%${search}%`,
}); });
const searchQuery = filters.search const searchQuery = filters.search
@ -46,7 +47,7 @@ async function relationalQuery(websiteId: string, filters: QueryFilters) {
${searchQuery} ${searchQuery}
order by created_at desc order by created_at desc
`, `,
{ ...queryParams, search: `%${search}%` }, queryParams,
filters, filters,
); );
} }