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.

Sign up for GitHub

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

Jump to bottom

Error when dismissing the scripts menu without selecting anything #24

Closed
PEZ opened this issue May 2, 2022 · 0 comments · Fixed by #47
Closed

Error when dismissing the scripts menu without selecting anything #24

PEZ opened this issue May 2, 2022 · 0 comments · Fixed by #47
Labels
bug Something isn't working

Comments

@PEZ
Copy link
Collaborator

PEZ commented May 2, 2022

Hitting escape in the scripts menu, prints an error message in the Joyride output channel:

ERROR: Run Workspace Script Failed:  The "path" argument must be of type string. Received null

So there is some unhandled case there.
@PEZ PEZ added the bug Something isn't working label May 2, 2022
PEZ added a commit that referenced this issue May 10, 2022
@PEZ
Check if we have picked script before running it
aa173f6 
Fixes #24
@PEZ PEZ mentioned this issue May 10, 2022
Merged
@PEZ PEZ closed this as completed in #47 May 10, 2022
PEZ added a commit that referenced this issue May 10, 2022
@PEZ
Check if we have picked script before running it (#47)
148d2af 
Fixes #24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check if we have picked script before running it BetterThanTomorrow/joyride
1 participant
@PEZ