Skip to content

Fix variable validation to show only one alert #466

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

Closed
wants to merge 1 commit into from

Conversation

felixerdy
Copy link
Member

Follow up of #465

Copy link

Tests

Test Name Status Flaky Duration
Blockly Editor Page Tests [Blockly] visits the editor page 968ms
Blockly Editor Page Tests [Blockly] visits the tutorial page 174ms
Blockly Editor Page Tests [Blockly] visits the gallery page 134ms
Blockly Editor Page Tests [Blockly] visits the faq page 121ms
Blockly Editor Page Tests [Blockly] visits the settings page 120ms
Blockly Editor Page Tests [Blockly] visits the login page 121ms
Blockly Editor Page Tests [Blockly] navigates to tutorial and back 14.0s
Blockly Editor Page Tests [Blockly] selects senseBox ESP 719ms
Blockly Editor Page Tests [Blockly] selects senseBox MCU 724ms
Blockly Editor Page Tests [Blockly] selects senseBox Mini 683ms
Blockly Editor Page Tests [Blockly] compiles code 19.9s
Code Editor Page Tests [CodeEditor] visits the code editor page 1.3s
Code Editor Page Tests [CodeEditor] selects senseBox ESP 457ms
Code Editor Page Tests [CodeEditor] compiles code 615ms
Code Editor Page Tests [CodeEditor] opens reset dialog 601ms
Code Editor Page Tests [CodeEditor] fetches libraries 2.8s

Github Test Reporter by CTRF 💚

@mariopesch
Copy link
Member

@felixerdy @georgi-s i think this PR superseeds this one with a simpler solution: #467

@mariopesch mariopesch closed this May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants