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 clientOnly not starting up after async startup added in 2.23 #3154

Merged
merged 1 commit into from
Jul 16, 2023

Conversation

sdetweil
Copy link
Collaborator

Fixes #3151

app.whenReady() was removed when async changes made, needed for clientonly in electron.js

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (develop@948910d). Click here to learn what that means.
The diff coverage is 0.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             develop    #3154   +/-   ##
==========================================
  Coverage           ?   25.62%           
==========================================
  Files              ?       53           
  Lines              ?    11492           
  Branches           ?        0           
==========================================
  Hits               ?     2945           
  Misses             ?     8547           
  Partials           ?        0           
Impacted Files Coverage Δ
clientonly/index.js 0.00% <0.00%> (ø)
js/electron.js 0.00% <0.00%> (ø)

@khassel khassel requested a review from rejas July 16, 2023 20:16
@rejas
Copy link
Collaborator

rejas commented Jul 16, 2023

could you point to the commit where it was removed? cant find it ATM

@sdetweil
Copy link
Collaborator Author

PR #3091

@sdetweil
Copy link
Collaborator Author

I commented it out in my test prototype

@khassel khassel merged commit 1bd146f into MagicMirrorOrg:develop Jul 16, 2023
@sdetweil sdetweil deleted the fix_client_startup branch July 15, 2024 18:08
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.

4 participants