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

Upgrade to Electron 11 #1895

Merged
merged 1 commit into from
Nov 18, 2020
Merged

Upgrade to Electron 11 #1895

merged 1 commit into from
Nov 18, 2020

Conversation

feross
Copy link
Member

@feross feross commented Nov 13, 2020

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[ ] New feature
[x] Other, please explain: Update dep

What changes did you make? (Give an overview)

Upgrade to Electron 11

Which issue (if any) does this pull request address?

Possibly addresses #1894

Is there anything you'd like reviewers to focus on?

I'd like to start testing out Electron 11 on master so we can see if there's any deprecated APIs that we need to upgrade, etc. Then when Electron 11 is stable, we can do another release of the app :)

UPDATE: Electron 11 stable was just released.

@feross feross changed the title Upgrade to Electron 11 Beta Upgrade to Electron 11 Nov 17, 2020
@feross
Copy link
Member Author

feross commented Nov 17, 2020

Electron 11 stable was just released. https://www.electronjs.org/releases/stable#release-notes-for-v1100

No breaking changes.

@feross feross merged commit ec0a1d1 into master Nov 18, 2020
@feross feross deleted the electron11 branch November 18, 2020 02:41
@feross
Copy link
Member Author

feross commented Nov 18, 2020

Self-merging without review -- sorry. I want to start testing Electron 11 so we're certain it's stable before the next release.

@hicom150
Copy link
Contributor

Sorry for the late review 😓

I've noticed that with electron@11 this error is thrown

electron.remote.screen.getAllDisplays is not a function (telemetry.js:77)

I've tested and this error does not appear with electron@10, so I suspect that could be something related with the warning that refers to electron/electron#23506

I'm trying to find a fix, but I've not find a solution yet 😓

@feross
Copy link
Member Author

feross commented Nov 18, 2020

I opened an issue to track this error, @hicom150. See #1896

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