Skip to content

Message search pagination #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kentcdodds wants to merge 2 commits into from
Closed

Message search pagination #29

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7e6a90c
Add search and pagination for past messages
cursoragent Jan 30, 2026
0c5b3ce
Fix past message search pluralization
cursoragent Jan 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
249 changes: 208 additions & 41 deletions app/routes/_app+/recipients+/$recipientId.past.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,48 +4,101 @@ import {
json,
type LoaderFunctionArgs,
} from '@remix-run/node'
import { useLoaderData } from '@remix-run/react'
import { Form, Link, useLoaderData, useSearchParams } from '@remix-run/react'
import { useId } from 'react'
import { GeneralErrorBoundary } from '#app/components/error-boundary.tsx'
import { Button } from '#app/components/ui/button.tsx'
import { Icon } from '#app/components/ui/icon.tsx'
import { Input } from '#app/components/ui/input.tsx'
import { Label } from '#app/components/ui/label.tsx'
import { requireUserId } from '#app/utils/auth.server.ts'
import { prisma } from '#app/utils/db.server.ts'

const PAGE_SIZE = 100

export async function loader({ params, request }: LoaderFunctionArgs) {
const userId = await requireUserId(request)
const recipientId = params.recipientId
invariantResponse(recipientId, 'Recipient id is required', { status: 400 })
const url = new URL(request.url)
const searchTerm = url.searchParams.get('search')?.trim() ?? ''
const pageParam = url.searchParams.get('page') ?? '1'
const parsedPage = Number.parseInt(pageParam, 10)
const requestedPage = Number.isNaN(parsedPage) ? 1 : parsedPage
const page = Math.max(1, requestedPage)
const recipient = await prisma.recipient.findUnique({
where: { id: params.recipientId, userId },
where: { id: recipientId, userId },
select: {
name: true,
phoneNumber: true,
messages: {
select: { id: true, content: true, sentAt: true, order: true },
orderBy: { order: 'asc' },
where: { sentAt: { not: null } },
},
},
})

invariantResponse(recipient, 'Not found', { status: 404 })

const { messages, ...recipientProps } = recipient
const baseMessageWhere = {
recipientId,
sentAt: { not: null },
}
const messageWhere = searchTerm
? {
...baseMessageWhere,
content: { contains: searchTerm },
}
: baseMessageWhere

const totalMessageCountPromise = prisma.message.count({
where: baseMessageWhere,
})
const filteredMessageCountPromise = searchTerm
? prisma.message.count({ where: messageWhere })
: totalMessageCountPromise

const [totalMessageCount, filteredMessageCount] = await Promise.all([
totalMessageCountPromise,
filteredMessageCountPromise,
])

const totalPages = Math.max(1, Math.ceil(filteredMessageCount / PAGE_SIZE))
const currentPage = Math.min(page, totalPages)
const pastMessages = await prisma.message.findMany({
where: messageWhere,
select: { id: true, content: true, sentAt: true },
orderBy: [{ sentAt: 'desc' }, { order: 'desc' }],
take: PAGE_SIZE,
skip: (currentPage - 1) * PAGE_SIZE,
})

const rangeStart =
filteredMessageCount === 0 ? 0 : (currentPage - 1) * PAGE_SIZE + 1
const rangeEnd = Math.min(currentPage * PAGE_SIZE, filteredMessageCount)

return json({
recipient: recipientProps,
messageCountDisplay: messages.length.toLocaleString(),
pastMessages: messages
.filter((m) => m.sentAt)
.sort((m1, m2) => m2.sentAt!.getTime() - m1.sentAt!.getTime())
.map((m) => ({
id: m.id,
sentAtDisplay: m.sentAt!.toLocaleDateString('en-US', {
weekday: 'short',
year: 'numeric',
month: 'short',
day: 'numeric',
hour: 'numeric',
minute: 'numeric',
}),
content: m.content,
})),
recipient,
messageCount: totalMessageCount,
messageCountDisplay: totalMessageCount.toLocaleString(),
filteredMessageCount,
filteredMessageCountDisplay: filteredMessageCount.toLocaleString(),
searchTerm,
pageInfo: {
currentPage,
totalPages,
rangeStart,
rangeEnd,
pageSize: PAGE_SIZE,
},
pastMessages: pastMessages.map((message) => ({
id: message.id,
sentAtDisplay: message.sentAt!.toLocaleDateString('en-US', {
weekday: 'short',
year: 'numeric',
month: 'short',
day: 'numeric',
hour: 'numeric',
minute: 'numeric',
}),
content: message.content,
})),
})
}

Expand All @@ -59,27 +112,141 @@ export const meta: MetaFunction<typeof loader> = ({ data }) => {

export default function RecipientRoute() {
const data = useLoaderData<typeof loader>()
const [searchParams] = useSearchParams()
const searchInputId = useId()
const hasSearch = data.searchTerm.length > 0
const hasMessages = data.filteredMessageCount > 0
const messageResultLabel = hasSearch
? data.filteredMessageCount === 1
? 'matching message'
: 'matching messages'
: data.filteredMessageCount === 1
? 'message'
: 'messages'

const buildPageLink = (page: number) => {
const params = new URLSearchParams(searchParams)
if (page <= 1) {
params.delete('page')
} else {
params.set('page', String(page))
}
const query = params.toString()
return query ? `?${query}` : '.'
}

const clearSearchParams = new URLSearchParams(searchParams)
clearSearchParams.delete('search')
clearSearchParams.delete('page')
const clearSearchLink = clearSearchParams.toString()
? `?${clearSearchParams.toString()}`
: '.'

return (
<div>
<p className="mb-8">
<div className="flex flex-col gap-6">
<p>
You have sent <strong>{data.messageCountDisplay}</strong>{' '}
{data.pastMessages.length === 1 ? 'message' : 'messages'} to{' '}
{data.messageCount === 1 ? 'message' : 'messages'} to{' '}
{data.recipient.name}.
</p>
<ul className="flex flex-col gap-2">
{data.pastMessages.map((m) => (
<li
key={m.id}
className="flex flex-col justify-start gap-2 align-top lg:flex-row"
>
<span className="min-w-36 text-muted-secondary-foreground">
{m.sentAtDisplay}
</span>
<span>{m.content}</span>
</li>
))}
</ul>
<div className="flex flex-col gap-4">
<Form method="GET" className="flex flex-wrap items-center gap-2">
<div className="flex-1">
<Label htmlFor={searchInputId} className="sr-only">
Search messages
</Label>
<Input
key={data.searchTerm}
type="search"
name="search"
id={searchInputId}
defaultValue={data.searchTerm}
placeholder="Search messages"
className="w-full"
/>
</div>
<input type="hidden" name="page" value="1" />
<Button type="submit" variant="secondary" size="sm">
<Icon name="magnifying-glass" size="md" />
<span className="sr-only">Search</span>
</Button>
{hasSearch ? (
<Button variant="ghost" size="sm" asChild>
<Link to={clearSearchLink} preventScrollReset>
Clear search
</Link>
</Button>
) : null}
</Form>
<div className="flex flex-wrap items-center justify-between gap-2 text-sm text-muted-foreground">
{hasMessages ? (
<p>
Showing {data.pageInfo.rangeStart}-
{data.pageInfo.rangeEnd} of{' '}
{data.filteredMessageCountDisplay} {messageResultLabel}
</p>
) : (
<p>
{hasSearch
? 'No messages match this search.'
: 'No messages have been sent yet.'}
</p>
)}
{data.pageInfo.totalPages > 1 ? (
<p>
Page {data.pageInfo.currentPage} of{' '}
{data.pageInfo.totalPages}
</p>
) : null}
</div>
</div>
{hasMessages ? (
<ul className="flex flex-col gap-2">
{data.pastMessages.map((message) => (
<li
key={message.id}
className="flex flex-col justify-start gap-2 align-top lg:flex-row"
>
<span className="min-w-36 text-muted-secondary-foreground">
{message.sentAtDisplay}
</span>
<span>{message.content}</span>
</li>
))}
</ul>
) : null}
{data.pageInfo.totalPages > 1 ? (
<div className="flex flex-wrap items-center justify-center gap-2">
{data.pageInfo.currentPage > 1 ? (
<Button variant="outline" size="sm" asChild>
<Link
to={buildPageLink(data.pageInfo.currentPage - 1)}
preventScrollReset
>
Previous
</Link>
</Button>
) : (
<Button variant="outline" size="sm" disabled>
Previous
</Button>
)}
{data.pageInfo.currentPage < data.pageInfo.totalPages ? (
<Button variant="outline" size="sm" asChild>
<Link
to={buildPageLink(data.pageInfo.currentPage + 1)}
preventScrollReset
>
Next
</Link>
</Button>
) : (
<Button variant="outline" size="sm" disabled>
Next
</Button>
)}
</div>
) : null}
</div>
)
}
Expand Down
Loading