-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Display language doesn't persist in code-server v3.11.0 running in Docker container when installing from CLI #3372
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
Comments
However, you can change the display language if you do the following:
ScreenshotAnd, if you restart code-server the language pack is installed and we set the locale in @aleleba can you please double-check that you have the language pack extension installed? |
@jsjoeio, Yes I can confirm that I install the language pack. Currently I install the Spanish Language package directly with a command in my Dockerfile. What happens in the previous versions of code-server is that when entering for the first time it already opened the Spanish language by default. I just tried uninstalling and installing the package again, restarted the window and changed the language. In the same way, I would like the language to be preloaded as before, so that if the Docker container falls, it takes the language configuration at once. I am attaching the docker docker command I use in the Dockerfile:
I really appreciate the follow-up and the help. |
Ah okay, thank you for all the details. So here's a screenshot of my running that command locally (not using Docker): Hmm...now this is interesting... When I look inside I'll keep both and restart code-server. According to VS Code's docs, Let's delete the Well, I'm a bit stumped. It feels like this is specific to how code-server is loading/running in your Docker environment. Maybe @code-asher has some insight. |
@jsjoeio, @code-asher, I started to see the changes from version 3.9.3 to version 3.10.0, to see if I can help to find the error and I could observe this change: https://github.com/cdr/code-server/pull/3250/files I think instead of placing:
You have to place:
To stand in a line. I don't use yarn and gulp, and I'm not an expert in bash, so I don't really know if that is the case. I hope to help a little |
Maybe @oxy might have some insight too. |
I was able to replicate with:
Strangely, I don't get the option to select If I uninstall the extension and re-install then refresh everything works. I think there is some language bundle generation that happens somewhere that isn't running when doing things in the CLI but is running when doing things through the UI. |
I just noticed another weird behavior in code-server. When deactivating an extension and reloading the window, this extension is not deactivated. I think this bug may be linked to that behavior in this version. |
no languagepacks.json found, It's what causes it not to work !!! when install language pack manually by UI,the generate |
Same here, languagepacks.json is missing after installing language pack in command line. |
I would like to follow up on this error. I would like to know what I can do to help solve it, I really like code-server, but I had to stay in version 3.9.3, where these errors do not occur, how can I help to solve this problem. A pleasure to greet the entire team. |
Thanks for offering to help @aleleba!
I think this would be the first place to start. Use the latest version of code-server and see if you can confirm this theory (i.e. can this issue be reproduced when doing things via the CLI and also through the UI?) |
I confirm this theory, when I installed the language from CLI no languagepacks.json found and when doing from IU it works |
Nice work! 🎉 Okay since this only happens with the CLI, my guess is we need to investigate there. Maybe we can write a function that checks if the extension to download is a language and if it is, add the languagepacks.json. Let's double-check with @code-asher though first P.S. - thanks for updating the title too! |
I'm not sure; we just shell straight out to VS Code to handle the
installation so it's odd that it doesn't work.
I suppose we should first check what desktop VS Code does. Install an
extension then (without launching VS Code!) see if the file exists.
If it does, that's weird and I'm not sure how to debug that but we'll
cross that bridge if we come to it.
If it doesn't, then launch VS Code and see if it appears then. If it
does then we might be missing something that is supposed to create the
missing file on startup/launch (much like we create missing directories,
etc). If this is the case then we'll probably want to look into how
desktop VS Code does it and copy/import that.
|
@jsjoeio, do think you can add that to 4.0.3 milestone? The function you mentioned already exists in VS Code API, it's For what I've seen, seems that |
@dgadelha unfortunately not since I'm at max bandwidth for that sprint, but I can add it as a high priority item. PRs are also welcome. Thanks for posting those notes! That will definitely speed things up when we get to this. |
which old version is ok? Or was it an existing BUG all along? |
I'm not sure 🤔 @code-asher might know if it previously worked |
I think it never worked. |
Can we make some following up @code-asher ? , If you need any help, it should be a pleasure to help. |
@aleleba can you try reproducing this with the latest version of code-server? |
Yes@jsjoeio, I already try and the behavior is the same. |
Thank you! If you want to dig in, I'd start here: #3372 (comment) |
I tried this with desktop VS Code ( |
This issue has been marked as 'upstream:vscode'. Please file this upstream: link to open issue |
1 similar comment
This issue has been marked as 'upstream:vscode'. Please file this upstream: link to open issue |
Thank you for your suggestion! Based on your idea, I wrote a Bash script to automatically generate the
With this setup, the |
OS/Web Information
code-server --version
:Version 3.11.0 of Code-Server
Steps to Reproduce
Expected
Change Languague Display
Actual
Not Change Language Display
Logs
Screenshot
Notes
This issue can be reproduced in VS Code: Yes/No
The text was updated successfully, but these errors were encountered: