Skip to content

Andrew/Search Chat Tickets By Name or Message Content - #355 - #361

Merged
ezraexe merged 2 commits into
mainfrom
Andrew/355-search-chat-tickets-by-name-or-message-content
Sep 14, 2026
Merged

ezraexe merged 2 commits into
mainfrom
Andrew/355-search-chat-tickets-by-name-or-message-content

Conversation

@aandrx

@aandrx aandrx commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Description

Resolves ticket number: #355

Explain what your code changes:
Added channel-list filters, extended to $or so it uses name.$autocomplete instead of just name, also partnerName. Also added new "createdByName" attribute that gets stored at ticket creation, so you can search by the name of the creator. Keep existing members.$in, so between Unresolved/Resolved tabs and partner isolation are maintained.
Add server endpoint that searches requested ticket IDs, and returns matched IDs that get shown in the results. Staff must have support access to use it (GET /api/tickets/search?q=)
Render filter matches title/partner/creator/member_names/last-message locally for the loaded channels, and added a pending text while searching for tickets by message.content. Search placeholder text also updated.
Add createdByName to be scored in createTicketChannel from the session user. Keep in mind that the pre-existing tickets lack this property, so they wont be able to be searched in this way.

List the steps you took to test your code:
Created 2 tickets for same partner (distinct titles+words), ran searches against both Super Admin and partner1, both tabs: ran title, partner-name, creator-name, message-prefix queries: confirmed isolation. Verified that clearing the search restores the full lists, pre-existing tickets without the new creator field still work (despite not being able to be searched that way).
End result:
Screenshot from 2026-09-11 14-54-07

Checklist

  • The ticket is mentioned above
  • The changes fulfill the success criteria of the ticket
  • The changes were self-reviewed
  • A review was requested

@netlify

netlify Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for hope-for-haiti ready!

Name Link
🔨 Latest commit 02bde56
🔍 Latest deploy log https://app.netlify.com/projects/hope-for-haiti/deploys/6aa61fd66cc42400087a3756
😎 Deploy Preview https://deploy-preview-361--hope-for-haiti.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@aandrx

aandrx commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

Before was 20 distinct messages then deduped, now its 20 distinct tickets, from scanning up to 60 newest messages.

image

@ezraexe

ezraexe commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

doesn't fully get rid of the issue, but raises the threshold significantly. i think this is fine for now given the current scale. can adjust as we go

@ezraexe
ezraexe merged commit bdaa66a into main Sep 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants