ioBroker.js-controller icon indicating copy to clipboard operation
ioBroker.js-controller copied to clipboard

[bug]: incorrect schema for io-package.json

Open oweitman opened this issue 1 year ago • 3 comments

No existing issues.

  • [X] There is no existing issue for my problem.

Description

the property ignoreInVersions is at the wrong position/hierychy

Reproduction instruction

see wrong/versus right object https://forum.iobroker.net/topic/71786/vis2-ab-morgen-im-stable/337?_=1719335514744

reproduction in vscode. there is a warning, what ignoreInVersions should be an object and not an array

no further errors in log

JS-controller version

6.05

Node version

v20.13.1

oweitman avatar Jun 25 '24 20:06 oweitman

I think we had this discussion started internally but not finished, we will take it into the next core refinement meeting.

foxriver76 avatar Jun 30 '24 19:06 foxriver76

Ok, The best thing would be to set the property at the widget set level, as it is currently defined in the schema. The advantage would be that within an adapter you could allow or prohibit a widget set for vis1 and a widget set for vis 2, etc.

oweitman avatar Jun 30 '24 19:06 oweitman

Yes, that was also my idea and why I started the discussion and did not change schema right away, will discuss with @GermanBluefox and @Apollon77 next week.

foxriver76 avatar Jun 30 '24 19:06 foxriver76

Ok, we will move it into the widget object, so schema is correct. Implementation will be adjusted next.

Apollon77 avatar Jul 10 '24 11:07 Apollon77

IF not yet working as intended please raise an issue at https://github.com/ioBroker/ioBroker.vis-2

foxriver76 avatar Sep 03 '24 13:09 foxriver76

cant find the commit where the issue is fixed in vis-2?

at the moment in npm the latest version is set to 2.9.64 Versions >2.10 cannot currently be tested regularly. only install per npm and explizit name the version. When will things continue here?

oweitman avatar Sep 03 '24 17:09 oweitman

Then please open the issue at vis 2 repo because implementation needs to be done there. Schema is correct as stated so nothing we can do in controller

foxriver76 avatar Sep 03 '24 18:09 foxriver76

also den kindergarten unterstütze ich nicht. das problem wurde erkannt, im devmeeting besprochen https://forum.iobroker.net/topic/75449/meeting-f%C3%BCr-iobroker-core-dev-admin-17-07-24-20-30?_=1725104975740 und von den devs entschieden, das die implementierung angepasst wird. es hätte ja auch sein können, das das schema angepasst werden muss. problem ist erkannt, ich glaube die todos sind auch bekannt. wenn es jetzt nur noch daran hängt, das der issue nicht im richtigen repo hängt???? du wärst doch auch der, der es anpassen könnte, neben bluefox?

dann lasst es so wie es ist. mein herz hängt da nicht dran. dann halt erst, wenn der nächste dev dann bereit ist das problem im richtigen repo zu erfassen.

oweitman avatar Sep 03 '24 18:09 oweitman

@oweitman nein bluefox muss es anpassen und was die reaction soll verstehe ich nicht. Du darfst auch gerne den Vis 2 PR erstellen

foxriver76 avatar Sep 03 '24 18:09 foxriver76