-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Error reported on the console after a task is completed and we re-select the terminal #5328
Comments
Another way to test, you can use the In tasks.json, replace by your environment: {
"label": "list all files in Theia packages",
"type": "shell",
"cwd": "${workspaceFolder}",
"command": "ls",
"args": [
"-alR",
"$<theia folder>/packages"
]
} |
Fixes #5328 Fix error thrown in the backend when after a task is completed. Instead of calling the method `getCwdURI(terminalId)` first check if the terminal with the given id truly exists. Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
Fixes #5328 Fix error thrown in the backend when after a task is completed. Instead of calling the method `getCwdURI(terminalId)` first check if the terminal with the given id truly exists. Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
After re basing With the latest commit (commit 741876b) this morning, I noticed an issue that was fixed a week ago |
To reproduce
|
@lmcbout are you sure the issue still exists? |
I still see it happen from my local env. |
Description
After executing a task which open a terminal and the task terminates, selecting another view in the bottom panel and back to the terminal, it generate the following error on the server console.
Reproduction Steps
Open the browser with Theia as a workspace
Open the problem or output view in the bottom panel
Select Run Task -> npm: test browser
Wait the task finish (A popup message will tell you when the task is completed
==> Select the output view then re--select the terminal view, the error should show on the server console log
OS and Theia version:
Theia version : 0.7.0
OS: Ubuntu 16.04
Browser: Chrome
Diagnostics:
The text was updated successfully, but these errors were encountered: