-
Notifications
You must be signed in to change notification settings - Fork 590
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
customized handlebars files can not be translated #6022
Comments
ok ive added a fix to my PR so you can use |
ok ive merged the PR, which means |
@si458 I have changed a lot of stuff in the default.handlebars file. I connect of the console of the docker container and run "meshcentral.js --translate" which generates translation in the mapped folder How can that be? Is the location not respected anymore? |
@cfoellmann do you have |
yes, it has produced those and those contain the "new" code |
have you tried ctrl+f5 and also restarting the meshcentral container? |
I had a look at my config.json and it is already Everything refreshed. ctrl+F5, browser cache, container restart, restartserver command. is there more? |
oh hang on ive just re-read ur message. so you changed will have alook into it! |
not really. it used to work. but now it wont pick up new changes. I am on the latest :master |
One more thing: I used the following command from within the container I will use the way described in #6168 |
And I have not refreshed the template from the master. That is a very bad merge since my git does not show my commit changes. probably because of the file size (Line count) |
The --translate operation exits with an error (presumably) after these lines:
I can only see the bottom half (I hope its half) of the error which comes down to this:
|
HUH? 😕 will try have a look tonight/2moz, as i havent changed anything for the translate since #6027 |
I did refresh the default.handlebars and did another |
I finally found the files that are being used: |
So you have not found a cause yet? |
no, it is doing minification despite the |
I am using a custom |
hello all, sorry for delay, been poorly again (wish my body would tell colds to do one and not return!) my findings on things
|
ok ive found the issue! going to take awhile to fix but im on it!
|
all fixed! #6180 if you want to look/apply the fix! |
basically its as explained above, it builds an array of files and languages from 'views' and 'meshcentral-web' and totally forgetting about 'meshcentral-web-domain'! |
the current :master is broken for me.
|
@cfoellmann just go into Check if If not do I will have another quick look as I did move the |
was not in the list. did a |
yeh its ok ive fixed the bug, doing a few merges of things and new builds! |
I had to also install jsdom for the
Is this normal? |
@cfoellmann ok fixed the connect-flash 7955bc4 as its needed in a few places, not just oidc! @DaanSelen yes thats perfectly normal, P.S both, just pushed new commit so new master images be ready soon for you to try with the language translate fixes in! |
where the --translate fixes not in? I tried a --translate and it still giving me the views from the custom domain folder :-( |
@cfoellmann try the new master image from 2 mins ago you might have to re-run the |
I was talking about the vulnerabilities introduces with this npm installation, |
@si458 working now. very much appreciated as always!! @DaanSelen I am pretty sure that most of the packages installed on the "start" of meshcentral would produce the "same" warnings: |
@DaanSelen sorry having a mad monday morning, mis-read message completely. p.s: dont forget to donate ❤️ https://www.si458.co.uk/2024/01/05/donation/ |
For me this issue still persists.
Is what I get when passing in |
@DaanSelen the must be some value or string inside your custom file that it just doesnt like!
|
Your fix for translating correctly. When I move it as backup file it works, but how can I troubleshoot where it errors? |
so if you renamed the file to its going to be very hard to work out what its not working? if your happy to, email me the file and ill have a quick dig https://github.com/si458 -> email on left! |
Also something, I don't know if this is related. But after removing the custom The Desktop and Terminal and files tab is not present. This should all be a default Meshcentral installation |
@DaanSelen Sorry for the delay, Mad day! |
modifying views for a domain in
meshcentral-web
ormeshcentral-web-domain2
do only work for language = enThe translation tool seems to not pick up these modified files and does not translate.
The text was updated successfully, but these errors were encountered: