Skip to content
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

[Feature Request]: comlink based IPC #413

Open
3 tasks done
ThaUnknown opened this issue Feb 3, 2024 · 1 comment
Open
3 tasks done

[Feature Request]: comlink based IPC #413

ThaUnknown opened this issue Feb 3, 2024 · 1 comment
Labels
development hell Can't make this work for shit enhancement New feature or request external This issue is caused by external dependencies, not the app itself.

Comments

@ThaUnknown
Copy link
Owner

Preflight checklist

  • I have searched the issue tracker for a bug report that matches the one I want to file, without success.
  • I have searched the features list for this feature, and I couldn't find it.
  • I have checked that I'm using the latest stable version of the app.

Problem Description

Current IPC sucks, like a lot, it's clunky annoying to expand

Proposed Solution

use comlink, adapt it to use message channels, event emitters and electron IPC

@ThaUnknown ThaUnknown added the enhancement New feature or request label Feb 3, 2024
@ThaUnknown
Copy link
Owner Author

limitations, limitations, the only way this can be built is without callbacks/transferables as both electron and capacitor struggle with them

@ThaUnknown ThaUnknown added development hell Can't make this work for shit external This issue is caused by external dependencies, not the app itself. labels Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development hell Can't make this work for shit enhancement New feature or request external This issue is caused by external dependencies, not the app itself.
Projects
None yet
Development

No branches or pull requests

1 participant