WIP: Fix bad network until a project is opened

[pierremtb]

Will fix #2610 and add tauri e2e test to check on the network indicator.

Merge #2659 first

[qa-wolf]

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
modeling-app	✅ Ready (Inspect)	Visit Preview	Jul 19, 2024 10:04pm

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.51%. Comparing base (34c5c15) to head (ede39b4).
Report is 3 commits behind head on main.

❗ Current head ede39b4 differs from pull request most recent head d9058c9

Please upload reports for the commit d9058c9 to get more accurate results.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2661      +/-   ##
==========================================
+ Coverage   86.54%   87.51%   +0.96%     
==========================================
  Files          60       56       -4     
  Lines       22823    22191     -632     
==========================================
- Hits        19752    19420     -332     
+ Misses       3071     2771     -300     

Flag	Coverage Δ
wasm-lib	87.51% <ø> (+0.96%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pierremtb]

Noticed that this does the trick only when you open the app signed in. Works after a refresh after signing in, which is probably why the test fails

[pierremtb]

Yeah forcing a refresh after auth makes the test happy.

@lf94 @franknoirot The way I tried to enable the engine connection on the home page here was a complete shot in the dark. Let me know how you'd want to proceed

[pierremtb]

Also, sorry for the dirty diff here. We should merge #2659 first

[lf94]

I can check this when Im home in 3hr. Still wip?

[pierremtb]

Well yeah still wip cause I have no idea if it’s the right solution, nor does it work first time after sign inOn Jun 16, 2024, at 4:59 PM, 49fl ***@***.***> wrote: I can check this when Im home in 3hr. Still wip? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[franknoirot]

Just merged #2659!

[pierremtb]

@franknoirot Thank you! Updating

[Irev-Dev]

Is this work still ongoing, or maybe we figured it was the wrong approach?

[pierremtb]

@Irev-Dev Not sure about the approach but I think we should keep the test. Kicking a branch update now

[jessfraz]

closed this was fixed elesewhere