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

refactor: remove dependency to deprecated remote module #593

Merged
merged 1 commit into from Jun 27, 2021

Conversation

csett86
Copy link
Member

@csett86 csett86 commented May 27, 2021

the remote module will be removed from electron 14 onwards,
so replace the locale detection with native browser api that
is available in the renderer.

Signed-off-by: Christoph Settgast csett86@web.de

the remote module will be removed from electron 14 onwards,
so replace the locale detection with native browser api that
is available in the renderer.

Signed-off-by: Christoph Settgast <csett86@web.de>
@csett86
Copy link
Member Author

csett86 commented May 29, 2021

@saghul FYI: I would merge these PRs myself (or directly push to master) after successful testing to help you with maintenance, but unfortunately thats rejected for contributors with the current permission setup for this repo.

@csett86 csett86 changed the title refactor: remove dependency to deprected remote module refactor: remove dependency to deprecated remote module Jun 1, 2021
@csett86
Copy link
Member Author

csett86 commented Jun 27, 2021

@saghul I have tested this successfully, please approve if you agree to this small deprecation fix.

@csett86 csett86 merged commit 2baa4b5 into master Jun 27, 2021
@csett86 csett86 deleted the remove-remote-module branch June 27, 2021 18:57
@csett86 csett86 mentioned this pull request Jun 30, 2021
d3473r pushed a commit to d3473r/jitsi-meet-electron that referenced this pull request Sep 1, 2023
the remote module will be removed from electron 14 onwards,
so replace the locale detection with native browser api that
is available in the renderer.

Signed-off-by: Christoph Settgast <csett86@web.de>
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.

None yet

2 participants