Rework DeployTool (4 Viewers)

[elliottmc]

With the new bin (in above post),

Where? I don't see it!

 

[Sebastiii]

Here the bin to test

 

[Sebastiii]

With the new bin (in above post),

Where? I don't see it!

I forgot to click on 'Submit' lol

 

[elliottmc]

Here the bin to test

Thanks. Not sure whether I can test this, since I can no longer reproduce the problem.

 

[elliottmc]

@Sebastiii - where are we with this?

Any more test results?

Want to commit the font fix to the branch and get more testing? Or simply ready to merge? The basic fix is certainly fine, and I'm not sure if the font issue is a result of this, or a separate issue.

 

[Sebastiii]

Hi Mark,

I just push the change i have made based on latest bin for testing, i can't reproduce it so i can't confirm it's 100% resolved, but the 'error' message seems to be a safe error message.

So i would say it's OK

 

[elliottmc]

Hi Mark,

I just push the change i have made based on latest bin for testing, i can't reproduce it so i can't confirm it's 100% resolved, but the 'error' message seems to be a safe error message.

So i would say it's OK

I have just updated two systems with the version in GIT without the font fix, and I didn't get the error on either. But if you can fix it for the people who see the problem and no regression for anyone else, then I'd say we are good.

What we now need is translations for the new strings.

 

[Sebastiii]

Yep need more translation and hope that the latest commit will solve the issue

 

[Sebastiii]

Installer on first post updated

 

[azzuro]

Seb, only missing option for keep setting on freshinstall