Skip to content

fix: use real timestamp for temp messages - #19025

Merged
Antreesy merged 3 commits into
mainfrom
fix/13777/temp-messages-timestamp
Aug 19, 2026
Merged

fix: use real timestamp for temp messages#19025
Antreesy merged 3 commits into
mainfrom
fix/13777/temp-messages-timestamp

Conversation

@Antreesy

Copy link
Copy Markdown
Contributor

☑️ Resolves

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

No visual changes

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • ⛑️ Tests are included or not possibleAssisted-by: ClaudeCode:claude-sonnet-5
    Signed-off-by: Maksim Sukharev antreesy.web@gmail.com

@Antreesy Antreesy added this to the ⛅ Next Beta/RC (35) milestone Aug 18, 2026
@Antreesy
Antreesy requested review from DorraJaouad and ShGKme August 18, 2026 15:10
@Antreesy Antreesy self-assigned this Aug 18, 2026
@Antreesy Antreesy added bug feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client labels Aug 18, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 18, 2026

@DorraJaouad DorraJaouad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hard to reproduce the issue, but the logic makes sense and everything previous as workaround was removed

@Antreesy
Antreesy force-pushed the fix/13777/temp-messages-timestamp branch from 011e282 to 146f453 Compare August 18, 2026 16:32
@Antreesy
Antreesy enabled auto-merge August 18, 2026 16:34
@Antreesy

Copy link
Copy Markdown
Contributor Author

/backport to stable34

Comment thread src/utils/message.ts
@Antreesy
Antreesy disabled auto-merge August 18, 2026 18:17
Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the fix/13777/temp-messages-timestamp branch from 146f453 to a47f2af Compare August 19, 2026 08:35
@Antreesy

Copy link
Copy Markdown
Contributor Author

Rebased, resolved conflicts
Renamed util to isTemporaryId
Migrated to convertToUnix() for other instances (mostly tests), no breaking changes

@Antreesy
Antreesy merged commit 6da34b7 into main Aug 19, 2026
54 checks passed
@Antreesy
Antreesy deleted the fix/13777/temp-messages-timestamp branch August 19, 2026 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits bug feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Temporary message added before date separator

5 participants