Skip to content

fix(threads): Add the thread title to the system message #15604

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

Jump to bottom
Merged
nickvergessen merged 2 commits into from
Jul 30, 2025
Merged

fix(threads): Add the thread title to the system message #15604

nickvergessen merged 2 commits into from
Jul 30, 2025

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Jul 30, 2025 •
edited
Loading

🛠️ API Checklist

  • Should allow clients to update the thread list from pulling the chat
    • Title is a parameter
    • Last activity the timestamp of the system message
    • num replies 0
    • last message null
    • first message the parent

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen added this to the 🪺 Next Major (32) milestone Jul 30, 2025
@nickvergessen nickvergessen self-assigned this Jul 30, 2025
@nickvergessen nickvergessen added 3. to review feature: chat 💬 Chat and system messages feature: api 🛠️ OCS API for conversations, chats and participants feature: threads 🧵 labels Jul 30, 2025
@nickvergessen
Copy link
Member Author

nickvergessen commented Jul 30, 2025

@nickvergessen nickvergessen requested a review from Ivansss July 30, 2025 08:12
Antreesy
Antreesy reviewed Jul 30, 2025
View reviewed changes
@nickvergessen nickvergessen force-pushed the bugfix/9679/thread-title-in-system-message branch from 20e5e93 to 2623698 Compare July 30, 2025 09:54
Antreesy
Antreesy approved these changes Jul 30, 2025
View reviewed changes
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested

Image

@nickvergessen nickvergessen merged commit 30fedba into main Jul 30, 2025
79 checks passed
@nickvergessen nickvergessen deleted the bugfix/9679/thread-title-in-system-message branch July 30, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Antreesy Antreesy Antreesy approved these changes

@miaulalala miaulalala Awaiting requested review from miaulalala

@Ivansss Ivansss Awaiting requested review from Ivansss

Assignees

@nickvergessen nickvergessen

Labels

3. to review feature: api 🛠️ OCS API for conversations, chats and participants feature: chat 💬 Chat and system messages feature: threads 🧵

Projects

None yet

Milestone

v22.0.0-beta.1

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen @Antreesy