Skip to content

Feat: Select welcome-select windows#540

Open
JavierRibaldelRio wants to merge 19 commits into
developfrom
control-station/multiple-select-app
Open

Feat: Select welcome-select windows#540
JavierRibaldelRio wants to merge 19 commits into
developfrom
control-station/multiple-select-app

Conversation

@JavierRibaldelRio

@JavierRibaldelRio JavierRibaldelRio commented Jun 15, 2026

Copy link
Copy Markdown
Contributor
image

@JavierRibaldelRio JavierRibaldelRio added Frontend javascript Pull requests that update javascript code labels Jun 15, 2026
@JavierRibaldelRio JavierRibaldelRio marked this pull request as ready for review June 15, 2026 23:32
@maximka76667

Copy link
Copy Markdown
Contributor

When I first open testing-view, then go back and open testing-view again application crashes

image

@maximka76667

Copy link
Copy Markdown
Contributor

You could consider adding a test that will verify this behavior in E2E and modifying e2e tests so they pass

// Start services and only then load the selected view.
if (view === "testing-view" || view === "flashing-view") {
await startServices(screenWidth, screenHeight, view);
await new Promise((resolve) => setTimeout(resolve, 1000));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure you need this timeout since you use async functions that are awaited

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even though startServices is awaited the testing view tries to connect to the backend before it has opened the websockets port. This extra delay at backend is caused by ADJ-Validator.

Without it does not work properly.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

At least it's supposed to connect only after the 'backend is ready' message arrives, but i'm not sure about details

Comment thread electron-app/src/ipc/handlers.js Outdated
@JavierRibaldelRio

Copy link
Copy Markdown
Contributor Author

When I first open testing-view, then go back and open testing-view again application crashes
image

Thats because is does not exists.

@JavierRibaldelRio

Copy link
Copy Markdown
Contributor Author

It seems I have messed up a little bit with GitHub, sorry

You could consider adding a test that will verify this behavior in E2E and modifying e2e tests so they pass

I'll look at it. ;)

@maximka76667

Copy link
Copy Markdown
Contributor

When I first open testing-view, then go back and open testing-view again application crashes
image

Thats because is does not exists.

i think you misread my message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants