Skip to content

fix: migrate fileSharingServices to TS #14586

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.

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

Merged
merged 4 commits into from
Mar 11, 2025
Merged

fix: migrate fileSharingServices to TS #14586

merged 4 commits into from
Mar 11, 2025

Conversation

Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Mar 7, 2025

☑️ Resolves

🖌️ UI Checklist

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Not risky to browser differences / client
  • ⛑️ Tests are included or not possible

@Antreesy Antreesy added this to the 🪺 Next Major (32) milestone Mar 7, 2025
@Antreesy Antreesy requested a review from DorraJaouad March 7, 2025 18:19
@Antreesy Antreesy self-assigned this Mar 7, 2025
Base automatically changed from fix/noid/openapi-files to main March 8, 2025 06:27
@Antreesy Antreesy force-pushed the fix/noid/file-share-ts branch 2 times, most recently from 3f52bf7 to 073fabb Compare March 10, 2025 09:46
Copy link
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.

Tested

@Antreesy Antreesy force-pushed the fix/noid/file-share-ts branch from 073fabb to d0d8219 Compare March 11, 2025 10:47
- add types
- minor refactoring

Signed-off-by: Maksim Sukharev <[email protected]>
- refactor shareFile method

Signed-off-by: Maksim Sukharev <[email protected]>
@Antreesy Antreesy force-pushed the fix/noid/file-share-ts branch from d0d8219 to 3ffadd2 Compare March 11, 2025 10:50
@Antreesy
Copy link
Contributor Author

/backport to stable31

@Antreesy Antreesy merged commit e50425c into main Mar 11, 2025
53 checks passed
@Antreesy Antreesy deleted the fix/noid/file-share-ts branch March 11, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants