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

Cloud: add handling of TDeviceConfig_PowerStatusLed #870

Closed
fracz opened this issue Sep 5, 2024 · 12 comments
Closed

Cloud: add handling of TDeviceConfig_PowerStatusLed #870

fracz opened this issue Sep 5, 2024 · 12 comments
Assignees

Comments

@fracz
Copy link
Member

fracz commented Sep 5, 2024

No description provided.

@fracz fracz self-assigned this Sep 5, 2024
@fracz
Copy link
Member Author

fracz commented Sep 11, 2024

hę?

@przemyslawzygmunt
Copy link
Member

Mam wątpliwości co do nazwy zmiennej w tej strukturze (natywnie) ale pomijając to, to jest to analogiczne do TDeviceConfig_StatusLed (statusLed).
json:

{"powerStatusLed":"DISABLED"}
{"powerStatusLed":"ENABLED"}

@klew
Copy link
Member

klew commented Sep 12, 2024

a jakbyś chciał to ją nazwać?

@przemyslawzygmunt
Copy link
Member

Nie pasuje mi PowerStatusLed"Type" do enabled/disabled.

@klew
Copy link
Member

klew commented Sep 12, 2024

To było dodane analogicznie do status led - tam są trzy opcje, a nie tylko enabled/disabled. Popraw wedle uznania :)

@przemyslawzygmunt
Copy link
Member

Może po prostu "Enabled" ?
1 true, 0 false

@klew
Copy link
Member

klew commented Sep 12, 2024

wolałbym aby 0 było wartością domyślmną i aby oznaczało "enabled". Jak będzie co innego, to trzeba będzie pilnować aby to dobrze ustawić

@przemyslawzygmunt
Copy link
Member

No to "Disabled"
1 true, 0 false

@fracz
Copy link
Member Author

fracz commented Sep 19, 2024

Nic nie rozumiem.

Czy w tym zadaniu chodzi o to, żeby dodać do user_config urządzenia wsparcie dla

{powerStatusLedDisabled: true} albo {powerStatusLedDisabled: false} + GUI?

@klew
Copy link
Member

klew commented Sep 19, 2024

Tak

@fracz
Copy link
Member Author

fracz commented Sep 23, 2024

image

@fracz fracz closed this as completed Sep 23, 2024
@przemyslawzygmunt
Copy link
Member

{"powerStatusLed":"DISABLED"}
{"powerStatusLed":"ENABLED"}

tak to działa. Dyskusja toczyła się w odniesieniu do struktury natywnej.

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

No branches or pull requests

3 participants