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

Fix issue with electron-store #87

Merged

Conversation

pelletier197
Copy link
Contributor

@pelletier197 pelletier197 commented Jul 15, 2024

This allows to use electron-store V10 inside the Renderer. Resolves #86

@pelletier197
Copy link
Contributor Author

Hey @caoxiemeihao! Sorry for tagging you, this is a bit a random PR but I have been experiencing an issue with electron store and this solved the issue for me. Do you see any problem with this solution?

Thank you 🙏

@Stanzilla
Copy link

@caoxiemeihao poke :)

@Stanzilla
Copy link

@pelletier197 ah if we also add BrowserWindow, would that fix #88 as well?

@pelletier197
Copy link
Contributor Author

Good question. Maybe, but i'm not sure BrowserWindow is supposed to be exposed in the renderer process. Maybe @caoxiemeihao would know ?

@caoxiemeihao
Copy link
Member

caoxiemeihao commented Sep 19, 2024

@Stanzilla @pelletier197 Sorry! I've been busy lately and I noticed this problem.
Perhaps we can list the modules exported by the Main process more completely. This will be more compatible.

Also, could you change the target branch to v0.14.6? This will be the next release.

@caoxiemeihao caoxiemeihao linked an issue Sep 19, 2024 that may be closed by this pull request
@caoxiemeihao caoxiemeihao merged commit d415315 into electron-vite:main Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Usage with electron-util Does not work with electron-store v10
3 participants