Skip to content

fix: add original sender to optimistic replies - #24

Closed
BP602 wants to merge 1 commit into
mainfrom
codex/fix-issue-12-in-kicktalk
Closed

BP602 wants to merge 1 commit into
mainfrom
codex/fix-issue-12-in-kicktalk

Conversation

@BP602

@BP602 BP602 commented Sep 3, 2025

Copy link
Copy Markdown
Owner

Summary

  • include original reply sender when constructing optimistic reply metadata so username appears immediately
  • pass complete metadata to Kick API

Testing

  • npm run lint
  • npm run build

https://chatgpt.com/codex/tasks/task_e_68b79af053f8833186f5b83f392cc5fe

@BP602 BP602 closed this Sep 3, 2025
ftk789 added a commit that referenced this pull request Oct 14, 2025
* Fix cross-platform icon compatibility

- Add cross-platform icon handling for Linux/macOS support
- Convert Windows .ico to .png for non-Windows platforms
- Update tray and thumbar icons to use platform-appropriate format

Fixes application crashes on Linux due to unsupported .ico format

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Implement draft message persistence between chatroom switches

- Add draft storage to ChatProvider state using Map for per-chatroom drafts
- Create DraftManager component to handle automatic saving and restoration
- Save draft content on editor changes using Lexical update listeners
- Restore draft content when switching back to chatrooms
- Clear drafts when messages are sent successfully
- Fix Lexical node structure by properly using paragraph nodes

Fixes #23

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: ftk789 <57598907+ftk789@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant