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

Implement a safety for leaving the script runner screen if a script is running #1630

Open
hmaclachlan opened this issue Oct 11, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@hmaclachlan
Copy link

Would it be possible to implement a safety feature or have a pop up appear on the screen if a user tries to exit the script runner while a script is in the middle of running?

For example, if I were running a script that had a "wait" or a long "wait_check" and clicked the telemetry viewer while the script was running in the same tab that I was running the script in, it appears as though the script terminates. Would it be possible to have a feature that prevents the user from opening another view within the same tab (but allows another tab to be opened)?

@jmthomas
Copy link
Member

No the script is still running. Just go back to Script Runner and select Script -> Execution Status. Then you can connect to the existing script. We could potentially add an "Are you sure" when navigating away from a running script but otherwise we want people to be able to navigate.

@jmthomas jmthomas added the question Questions about OpenC3 label Oct 11, 2024
@hmaclachlan
Copy link
Author

Thanks for the clarification on that - an "are you sure" would be nice, but as long as the script is still running the execution status is all I need.

@jmthomas jmthomas added enhancement New feature or request and removed question Questions about OpenC3 labels Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants