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

Migration to jsonConfig #559

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Jey-Cee
Copy link
Member

@Jey-Cee Jey-Cee commented Oct 10, 2024

No description provided.

@Jey-Cee
Copy link
Member Author

Jey-Cee commented Oct 10, 2024

XS

grafik
grafik
grafik

SM

grafik
grafik
grafik

MD

grafik
grafik
grafik

LG

grafik
grafik
grafik

XL

grafik
grafik
grafik

@mcm1957 mcm1957 requested a review from Apollon77 October 14, 2024 17:54
@mcm1957 mcm1957 changed the title Json config Migratio n to jsonConfig Oct 14, 2024
@mcm1957 mcm1957 changed the title Migratio n to jsonConfig Migration to jsonConfig Oct 14, 2024
@mcm1957
Copy link
Member

mcm1957 commented Oct 14, 2024

@Apollon77
@GermanBluefox

As registered maintainers (https://github.com/iobroker-community-adapters/info#maintainers) please review / merge and create a new release if OK.

@simatec
Copy link
Contributor

simatec commented Oct 15, 2024

Please added the width for the Tab after "type": "tabs",

"tabsStyle": {
        width: "calc(100% - 100px)"
    },

@mcm1957
Copy link
Member

mcm1957 commented Oct 15, 2024

Please added the width for the Tab after "type": "tabs",

"tabsStyle": {
        width: "calc(100% - 100px)"
    },

Looks like you suggest to add this to any table?
If this is used regulary why can't this be added as a default?

@Jey-Cee
Copy link
Member Author

Jey-Cee commented Oct 15, 2024

@simatec this seems not to make a difference. Can you explain why this should be added?

@simatec
Copy link
Contributor

simatec commented Oct 15, 2024

@simatec this seems not to make a difference. Can you explain why this should be added?

This eliminates the overlap with the arrow buttons when the view is on very small devices

image

Example:
image

@mcm1957
Copy link
Member

mcm1957 commented Oct 15, 2024

@simatec
kannst du mit BF kläten ob man das nicht defaulten könnte? Ist irgendwie fad das in jede Tabelle aufs Neue reinzukopieren. Wiederspricht ein wenug dem jsonCongig gedanjen.

@simatec
Copy link
Contributor

simatec commented Oct 15, 2024

Grundsätzlich macht es mehr Sinn, auch in der jsonConfig ein Hamburger Menü zu erzeugen bei kleinen Bildschirmen. Das würde ich mit @GermanBluefox mal besprechen

@Jey-Cee
Copy link
Member Author

Jey-Cee commented Oct 15, 2024

Grundsätzlich macht es mehr Sinn, auch in der jsonConfig ein Hamburger Menü zu erzeugen bei kleinen Bildschirmen. Das würde ich mit @GermanBluefox mal besprechen

Dann lass uns darauf hinarbeiten und nicht in jedem Adapter diesen Style hinzufügen. Hineterher müssen wird das sonst noch bei allen wieder ausbauen weil es irgendwas Kaputt macht mit dem Burger Menü.

@simatec
Copy link
Contributor

simatec commented Oct 16, 2024

Ich vermisse in der jsonConfig den Button "Geräte suchen"
image

@Jey-Cee
Copy link
Member Author

Jey-Cee commented Oct 16, 2024

Ich vermisse in der jsonConfig den Button "Geräte suchen" image

Ah erwischt, den hatte ich Unterschlagen.
Leider lässt der sich nicht einfach ersetzen oder ich weis nur nicht wie.
Wir haben ja keinen Button mit Popup oder Modal.

EDIT: Das wäre jetzt eigentlich ein Fall für den Device Manager. Dafür wären aber Grundlegende Änderungen nötig, zu viel um sie jetzt schnell zu machen.

@simatec
Copy link
Contributor

simatec commented Oct 16, 2024

Dazu wird vergleichbar dem Ping Adapter ein customComponents benötigt... Also nur mit react möglich

@mcm1957
Copy link
Member

mcm1957 commented Oct 16, 2024

Warum geht das nicht via sendTo o.ä.? Suchen wird doch zu 99% eh der Adaptercode und nicht das Frontend. Starte suche per Button und Code trägt die gefundenen Geräte in Config ein. Reload und fertig. Oder eh ich da was falsch?

Custom Components sollten m.E wenn irgend möglich vermieden werden. Damit wird in einen an sich einfach zu wartenden Adapter REACT eingeschleppt dass dann nur 2 bis 3 Leute warten können. Wenn eine neue Funktion sinnvoll ist, dann bitte eine neue Component für jsonConfig erstellen / anregen.

Ich hatte schon 2 od. 3 mal das Problem einen einfachen Adapetr nicht aktualisieren zu können weil das Sch... react klemmt.

@Jey-Cee
Copy link
Member Author

Jey-Cee commented Oct 16, 2024

In dem Fall ist es so das die Suche eine Auswahl öffnet, aus der man dann Wählen kann welche Geräte man hinzufügen will.

@mcm1957
Copy link
Member

mcm1957 commented Oct 16, 2024

Varianten:
-) man legt eine standard jsonConfig Komponente an die das kann
-) man wartet auf den Devicemanager (der so was ja können sollte)
-) man trägt alle gefundenen Geräte in eine / die Tabelle ein und setzt ein flag active auf falsch. User kann dann dort aktivieren oder das Gerät aus der Tabelle löschen

Alles wartbarer als eine jsonConfig Spezialkomponente in REACT. Wir müssen wirklich beachten, dass außer BF kaum jemand in React halbwegs sattelfest ist und auch nur bei einem Bauproblem sagen kann wo es denn klemmen könnte.

Im Grunde kann ich mir auch gut vorstellen bei diesem Adapter die Gerätesuche gleich wegzulassen. Wer hat denn schon mehrere tr064 Geräte? Da sind ja mehrere Geschirrspüler ebenso wahrscheinlich.

@simatec
Copy link
Contributor

simatec commented Oct 16, 2024

Im Grunde kann ich mir auch gut vorstellen bei diesem Adapter die Gerätesuche gleich wegzulassen. Wer hat denn schon mehrere tr064 Geräte?

Der Adapter sucht IP Geräte

@simatec
Copy link
Contributor

simatec commented Oct 16, 2024

Alles wartbarer als eine jsonConfig Spezialkomponente in REACT. Wir müssen wirklich beachten, dass außer BF kaum jemand in React halbwegs sattelfest ist und auch nur bei einem Bauproblem sagen kann wo es denn klemmen könnte.

Wir werden bei speziellen Anforderungen nicht um React drum rum kommen.

@mcm1957
Copy link
Member

mcm1957 commented Oct 16, 2024

Dann sollten wir die Umstellung solcher Adapter 3x überlegen. REACT macht das Ding in meinen Augen unwartbar. Zumindest solange nicht mehr REACT Programmierer verfügbar sind.

@mcm1957
Copy link
Member

mcm1957 commented Oct 16, 2024

Ah erwischt, den hatte ich Unterschlagen. Leider lässt der sich nicht einfach ersetzen oder ich weis nur nicht wie. Wir haben ja keinen Button mit Popup oder Modal.

EDIT: Das wäre jetzt eigentlich ein Fall für den Device Manager. Dafür wären aber Grundlegende Änderungen nötig, zu viel um sie jetzt schnell zu machen.

Ich habe bei BF schon beim letzten Meeting angeregt in der jsonConfig ein zur Laiufzeit konfigurierbares PopUp umzusetzen - als Standardkomponte:

ioBroker/ioBroker.admin#2672

Ev. waär es sinnvoll das in Releation zu dieser Anforderunge hier durchzudenken

@simatec
Copy link
Contributor

simatec commented Oct 16, 2024

Dann sollten wir die Umstellung solcher Adapter 3x überlegen. REACT macht das Ding in meinen Augen unwartbar. Zumindest solange nicht mehr REACT Programmierer verfügbar sind.

Hmmm woran machst du das genau fest? Ich sehe das ein wenig anders. Klar kann es sein, dass ein Build mal fehlschlägt, da sich in der adapter-react etwas geändert hat. Aber im großen und ganzen sehe ich am Beispiel von Backitup eigentlich sehr viele Vorteile mit React. Ich musste mich durch Backitup damit beschäftigen und würde bei weiten nicht behaupten, dass sich Sattelfest bin. Aber grundsätzlich denke ich, ist React am Ende einfacher als eine index_html, wo du extrem viel mit Frontend und CSS zu tun hast.
Aber ja ich gebe dir recht... man muss sich die Zeit nehmen und sich damit intensiv auseinander setzen.

@mcm1957
Copy link
Member

mcm1957 commented Oct 16, 2024

Dann sollten wir die Umstellung solcher Adapter 3x überlegen. REACT macht das Ding in meinen Augen unwartbar. Zumindest solange nicht mehr REACT Programmierer verfügbar sind.

Hmmm woran machst du das genau fest? Ich sehe das ein wenig anders. Klar kann es sein, dass ein Build mal fehlschlägt, da sich in der adapter-react etwas geändert hat. Aber im großen und ganzen sehe ich am Beispiel von Backitup eigentlich sehr viele Vorteile mit React. Ich musste mich durch Backitup damit beschäftigen und würde bei weiten nicht behaupten, dass sich Sattelfest bin. Aber grundsätzlich denke ich, ist React am Ende einfacher als eine index_html, wo du extrem viel mit Frontend und CSS zu tun hast. Aber ja ich gebe dir recht... man muss sich die Zeit nehmen und sich damit intensiv auseinander setzen.

Eine index_m.html die einmal funktioniert blockiert keinen Bauvorgang / Release wenn man nichts dran ändert.
React ist nach meiner Erfahrung immer ein "seh ma mal".

Und das Problem ist nicht react an sich. Wenn sich ein DEV entscheidet bei SEINEM Adapter react zu verwenden kann er das gerne. Aber solange es so wenige Devs gibt die sich mit React auskennen - incl. der ganzen Bauumgebung - sollten wir das so sparsam wie möglich einsetzen. Ist meine Meinung. Ich werd dich gern kontaktieren wenn der nächste React Bau klemmt.

Wird aber hier dann zu offtopic.

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

Successfully merging this pull request may close these issues.

3 participants