Skip to content

fix: migrate uploadStore to Pinia 🍍 #17172

Merged
Antreesy merged 3 commits intomainfrom
fix/noid/upload-store-ts
Feb 20, 2026
Merged

fix: migrate uploadStore to Pinia 🍍 #17172
Antreesy merged 3 commits intomainfrom
fix/noid/upload-store-ts

Conversation

@Antreesy
Copy link
Copy Markdown
Contributor

@Antreesy Antreesy commented Feb 19, 2026 •
edited
Loading

☑️ Resolves

  • Ref somewhere

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

No visual changes

🚧 Tasks

  • Manual testing
  • Better Typescript support (may need to rework store after migration, as we're messing with other stores and types too much)
  • Improve test coverage (may need to rework store after migration, as we're messing with other stores and types too much)
upload.ts |  78.06 | 107-118,189,247,414-427,434-438,470-471,476-477,505,551-554,573-582
  • Squash after review

🏁 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 possible

@Antreesy Antreesy added this to the 🏖️ Next Major (34) milestone Feb 19, 2026
@Antreesy Antreesy self-assigned this Feb 19, 2026
@Antreesy Antreesy added technical debt feature: frontend 🖌️ "Web UI" client feature: upload & shares & voice 📤🎙️ Sharing files into a chat and audio recordings labels Feb 19, 2026
@Antreesy Antreesy force-pushed the fix/noid/upload-store-ts branch from 98162a7 to 751e903 Compare February 19, 2026 17:28
DorraJaouad
DorraJaouad approved these changes Feb 20, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

looks clean

@Antreesy
fix(uploadStore): rename fileUploadStore.js to upload.ts
6e74e81 
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
fix(uploadStore): migrate upload.ts store to Pinia
386bd5b 
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
fix(uploadStore): adjust component imports and tests
682e4e8 
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the fix/noid/upload-store-ts branch from 751e903 to 682e4e8 Compare February 20, 2026 14:38
@Antreesy Antreesy merged commit cf0d3ca into main Feb 20, 2026
57 checks passed
@Antreesy Antreesy deleted the fix/noid/upload-store-ts branch February 20, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DorraJaouad DorraJaouad DorraJaouad approved these changes

@ShGKme ShGKme Awaiting requested review from ShGKme

Assignees

@Antreesy Antreesy

Labels

feature: frontend 🖌️ "Web UI" client feature: upload & shares & voice 📤🎙️ Sharing files into a chat and audio recordings technical debt

Projects

None yet

Milestone

v24.0.0-beta.1

Development

Successfully merging this pull request may close these issues.

2 participants

@Antreesy @DorraJaouad