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.

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

Current version of instruction for js-controller update is not shown #1311

Closed
Feuer-sturm opened this issue Feb 6, 2022 · 17 comments
Closed
Assignees

Comments

@Feuer-sturm
Copy link
Collaborator

Describe the bug
If you click on the button to show in the host settings the instructions how to update js-controller an older version of the instruction will be shown:
grafik

To Reproduce
Steps to reproduce the behavior:

  1. Go to Host settings
  2. Click on the button to show the instruction how to update js-controller

Expected behavior
The expectation is that the current text of this page https://github.com/ioBroker/ioBroker.docs/blob/master/admin/de/controller-upgrade.md will be shown in the update instruction dialog window

Versions:

  • Admin 5.2.3
  • js-controller 4.0.3
@oweitman
Copy link

oweitman commented Feb 8, 2022

additional a missing space between bash and -
image

see also:
https://forum.iobroker.net/topic/52208/js-controller-4-0-jetzt-im-beta-latest/134?_=1644324912184

@Feuer-sturm
Copy link
Collaborator Author

@oweitman
Wobei hier auch die Ursache sein kann, das gar nicht die aktuelle Version von dieser Seite angezeigt wird https://github.com/ioBroker/ioBroker.docs/blob/master/admin/de/controller-upgrade.md angezeigt wird, sondern irgendwas altes. Vielleicht ist dort auch in der alten Version der Fehler. Wird man vermutlich erst dann prüfen können, wenn die korrekte Version wieder im Admin angezeigt wird.

@oweitman
Copy link

oweitman commented Feb 8, 2022

in meiner Testbeschreibung unter dem link kannst du lesen, das ich einen extra dockercontainer frisch aufgesetzt habe und alles aktuell geladen wurde. wenn jemand ebenfalls neu installiert oder ein upgrade macht, dann müsste er das selbe sehen.
Hintergrund waren die kompatibilitätschecks für meine adapter

@Feuer-sturm
Copy link
Collaborator Author

@oweitman Meine Antwort sollte eine mögliche Erklärung sein, warum kein Leerzeichen vor dem Bindestrich beim dem Linux Kommando ist. In der Github Seite, aus welcher die Dokumentation/ der Inh Inhalt vom Screenshot stammt ist das Leerzeichen vorhanden.

@GermanBluefox
Copy link
Contributor

Changed in 5.3.0

@Feuer-sturm
Copy link
Collaborator Author

Issue is fixed with Admin 5.3.0

Unix command is shown correctly as well
grafik

@Feuer-sturm
Copy link
Collaborator Author

Feuer-sturm commented Feb 25, 2022

@GermanBluefox With current admin version 5.3.1 the old version of the instruction "Anweisungen zum Upgrade des js-controllers" is shown once again and not the current one.

@klein0r
Copy link
Contributor

klein0r commented Feb 25, 2022

Unix command is shown correctly as well

Is that still correct? What's the difference to iob fix ?

@Apollon77
Copy link
Collaborator

iob fix exists since a special point in time of the installer ... so depending on when fixer/installer was last used it will exist of you need to execute the shell command (in fact iob fix will also only execute this shell command)

@Apollon77
Copy link
Collaborator

@GermanBluefox Plan was to always load the content dynamicaly from e.g. https://github.com/ioBroker/ioBroker.docs/blob/master/admin/de/controller-upgrade.md ... only if this is not available we should show an "static variant" (ideally dynamicylly added on release of each admin version

@mcm1957
Copy link
Contributor

mcm1957 commented Mar 7, 2022

I've encountered the same problem. See https://forum.iobroker.net/topic/53153/update-js-controller-verbesserungsw%C3%BCrdige-anleitung?_=1646680822191

In addtion I would like to inform a "problem" which hit me as a very new user.

I spent noticeable time to realize what the "1" near the HOST symbol tries to indicate. And I could not realize what the version number at line named identical to my hostname refers to. I assumed thats a version of Raspion OS or so. It took some time that I realized, thats the js-controller version.

the "problem" was taht as all forum topics and at github we talk about js-controller, js-controller upgrade and so an. But the host panel does not mention the name js-controller anywhere.

I would suggest to evaluate whether the wording js-controller / jscontroller-version should be added at the hist panel at some place. For me theres no more problem of course but I assume, that other new users might be confused too. So please read this as a feedback of a new user.

Note: German verion of this feedback is present at above forum thread.

McM

@Feuer-sturm
Copy link
Collaborator Author

Admin 6.1.6
With current admin version there is still the old static update instruction visible and not the content from https://github.com/ioBroker/ioBroker.docs/blob/master/admin/de/controller-upgrade.md

@Feuer-sturm
Copy link
Collaborator Author

Feuer-sturm commented Jun 29, 2022

@GermanBluefox
Admin 6.1.7

ok

The current instruction is now shown :-)

not ok

  1. If you are pressing the button "In die Zwischenablage kopieren" there is a little popup "copied" but if I insert the content in a text editor there is nothing from the instructions. There is the same error if I click on the little copy item for one command
  2. Would it be better to have a copy icon after each command like you implemented it for single commands (2nd screenshot). If you have only one button the comments for some commands needs to removed. This can be avoid if there is a copy icon for each command
  3. Is it possible to "highlight" in some way the commands so that it is easier to differ between text and command?

grafik
grafik

@Feuer-sturm Feuer-sturm removed the fixed label Jun 29, 2022
@GermanBluefox
Copy link
Contributor

Which browser is that? Chrome and FF tested = >OK
admin

@GermanBluefox
Copy link
Contributor

image

@Feuer-sturm
Copy link
Collaborator Author

@GermanBluefox
I made my tests with the not working copy pastewith Firefox and Chrome with Admin 6.1.7

Admin 6.1.8
Everything is fine and working :-) Thank you.

@mcm1957 @GermanBluefox
As a feedback to your suggestion to write some explantion about js-controller and "Host".
I made proposal in a pullrequest https://github.com/ioBroker/ioBroker.docs/pull/390/files to add some explanation to the update instruction.
Perhaps @GermanBluefox has a good idea where in the "Host menu" can be a hint that the shown version is related to the js-controller

@Feuer-sturm
Copy link
Collaborator Author

Admin 6.1.9 everything is fine.

@mcm1957
Bluefox added for the version number the hint to js-controller :-)
grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants