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 electron 14 #1763

Closed

Conversation

PikachuEXE
Copy link
Collaborator

@PikachuEXE PikachuEXE commented Sep 30, 2021

Pull Request Type
Please select what type of pull request this is:

  • Bugfix
  • Feature Implementation

Related issue
N/A

Description
Upgrade electron 13.x > 14.x AND fix the issue caused by a breaking change in electron 14

Breaking change:
electron/electron#28550
Child windows no longer inherit BrowserWindow construction options from their parents.

Screenshots (if appropriate)
Nothing visible should really changed

Testing (for code that is not small enough to be easily understandable)
All Dev team members should test this in

  • Multiple OS
  • Dev version (npm run dev) and built version afterward (can trigger nightly build manually)

Desktop (please complete the following information):

  • OS: MacOS
  • OS Version: 11.6
  • FreeTube version: based on 7d0880a

Additional context
The upgrade had caused issue like #1636

References:

@PrestonN PrestonN enabled auto-merge (squash) September 30, 2021 03:38
@peepo5
Copy link
Contributor

peepo5 commented Sep 30, 2021

#1640 needs to be fixed if this is added

@peepo5 peepo5 added the PR: WIP label Sep 30, 2021
@peepo5
Copy link
Contributor

peepo5 commented Sep 30, 2021

Added WIP because there are still remaining issues

@PikachuEXE
Copy link
Collaborator Author

For #1640
This is how it looks if upgrade to electron 15.x

image

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

This will fix this #1768 (comment)
Also check the patch notes of this release u'll see it listed in it

Edit: I was wrong this was fixed in v13.5.1 and has been implemented in the RC.

@efb4f5ff-1298-471a-8973-3d47447115dc

To be honest i would just close this and go for the other PR. I've tested it extensively and everything seems to work fine.

@PikachuEXE
Copy link
Collaborator Author

#1764 should close this if that one is merged
This is the backup plan if any significant issue found and hard be fixed

Breaking change:
electron/electron#28550
Child windows no longer inherit BrowserWindow construction options from their parents.
auto-merge was automatically disabled November 10, 2021 14:51

Pull request was closed

@PikachuEXE PikachuEXE deleted the upgrade/electron-14 branch November 15, 2021 02:38
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: dependencies Pull requests that update a dependency file and removed PR: WIP electron being electron Electron related bugs and trickeries labels Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants