Skip to content

feat: scheduled message API #16297

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
miaulalala merged 5 commits into from
Dec 9, 2025
Merged

feat: scheduled message API #16297

miaulalala merged 5 commits into from
Dec 9, 2025

Conversation

@miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Nov 10, 2025 •
edited
Loading

☑️ Resolves

API part for #3954

🛠️ API Checklist

🚧 Tasks

  • create new scheduled message
    • test
  • update a scheduled message
    • sendAt
    • message text
    • If scheduled thread, thread title
    • silent
    • test
  • get all scheduled messages per room
    • test
  • delete a scheduled message
    • test
  • get a single scheduled message - remove
  • Add hasSchedulingMessages to attendee
  • Expose hasScheduledMessages property on Room

Follow ups:

Moved to #16508

🏁 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

@miaulalala miaulalala added this to the 🍏 Next Major (33) milestone Nov 10, 2025
@miaulalala miaulalala self-assigned this Nov 10, 2025
nickvergessen
nickvergessen reviewed Nov 11, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Nov 11, 2025
View reviewed changes
SystemKeeper
SystemKeeper reviewed Nov 20, 2025
View reviewed changes
miaulalala
miaulalala commented Nov 20, 2025
View reviewed changes
@miaulalala miaulalala force-pushed the feat/3954/scheduled-messages branch from 8eb9fe8 to 6512845 Compare December 1, 2025 11:34
nickvergessen
nickvergessen reviewed Dec 2, 2025
View reviewed changes
@miaulalala miaulalala force-pushed the feat/3954/scheduled-messages branch from 1ef7a81 to f0ebdc0 Compare December 2, 2025 16:06
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
@nickvergessen
Copy link
Member

nickvergessen commented Dec 3, 2025 •
edited by miaulalala
Loading

  • If we expose the attendee has-scheduled-messages in the roomformatter, clients could implement a filter option in the future

@miaulalala miaulalala force-pushed the feat/3954/scheduled-messages branch from 09e7a2d to 3d1c868 Compare December 3, 2025 12:40
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 3, 2025
View reviewed changes
Antreesy
Antreesy reviewed Dec 4, 2025
View reviewed changes
Antreesy
Antreesy reviewed Dec 4, 2025
View reviewed changes
@miaulalala miaulalala requested a review from Antreesy December 4, 2025 13:57
@Antreesy

This comment was marked as resolved.

Sign in to view
nickvergessen
nickvergessen reviewed Dec 5, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 5, 2025
View reviewed changes
nickvergessen
nickvergessen reviewed Dec 5, 2025
View reviewed changes
Antreesy
Antreesy approved these changes Dec 9, 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.

Also tested from my side, feels good)

miaulalala and others added 5 commits December 9, 2025 12:54
@miaulalala
feat: scheduled message API
ef3265c 
Signed-off-by: Anna Larch <anna@nextcloud.com>
@nickvergessen @miaulalala
fix(scheduled): Fix attendee has_scheduled_messages
92a9e51 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @miaulalala
fix(scheduled): Remove roomId and parentId from response and make id …
494ce57 
…a string

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @miaulalala
fix(scheduled): Fix response format
ae0c199 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @miaulalala
chore(assets): Recompile assets
55b6555 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@miaulalala miaulalala force-pushed the feat/3954/scheduled-messages branch from 4375304 to 55b6555 Compare December 9, 2025 11:57
@miaulalala miaulalala marked this pull request as ready for review December 9, 2025 11:57
@miaulalala miaulalala merged commit e91534f into main Dec 9, 2025
83 checks passed
@miaulalala miaulalala deleted the feat/3954/scheduled-messages branch December 9, 2025 12:10
@sowjanyakch sowjanyakch mentioned this pull request Dec 10, 2025
Closed
@SystemKeeper SystemKeeper mentioned this pull request Jan 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@Antreesy Antreesy Antreesy approved these changes

@SystemKeeper SystemKeeper Awaiting requested review from SystemKeeper

Assignees

@miaulalala miaulalala

Labels

enhancement feature: chat 💬 Chat and system messages

Projects

None yet

Milestone

v23.0.0-beta.1

Development

Successfully merging this pull request may close these issues.

4 participants

@miaulalala @nickvergessen @Antreesy @SystemKeeper