KLOP, KPOP..... KTOP ?

[klew]

Absolutnie nie, trzeba czasem jednak pracować albo ogarniać koło domu

Podaj adres, przyślemy chłopaków do oganiania koło domu, abyś mógł zająć się kodem na spokojnie

[[email protected]]

Temat trawnika to szkoda gadać, nic tu nie pomoże...

Supla-Cloud:
https://github.com/SUPLA/supla-cloud/pull/1068

Supla-Core:
https://github.com/SUPLA/supla-core/pull/609

SuplaDevice:
https://github.com/SUPLA/supla-device/pull/171

Proponowane zmiany:
* w bazie danych,
* w obsłudze kanału, jak np. historia, możliwość jej pobrania
* w UI
* w reakcjach,
* w powiadomieniach
* w API
* w MQTT - %email%/supla/17852f1c451f9e53091a795f24b97617/devices/30/channels/563/state/value == SUPLA GPT channel works!
* w SuplaDevice

Kanał jest jednokierunkowy, wyświetla tylko stringa, pozwala na historie, która jest wyświetlana jako tabelka.

Kilka screenów - fajna praktyka w moich projekcie, wrzucanie screenów z UI, żeby każdy kto nie renderuje w głowie mógł zobaczyć czego dotyczą zmiany w UI

gpt1.png

Linter errors: 80 znaków w linijce , wasz kod, wasze reguły, ale chętnie się dowiem jak to ma wyglądać inaczej jak tyle znaków wychodzi, mogę oczywiście skrócić np. ten define:

Code: Select all

  bool same = (strncmp(this->text, text, SUPLA_GENERAL_PURPOSE_TEXT_MAX_SIZE) == 0);

ale no tu?

Code: Select all

  } else if (is_condition_met(op, condition_met_at.tv_sec ? saved_old_text : old_value,
                              new_value, text_value)) {

@pzygmunt mógłbyś sprawdzić czy wszystko jest OK z podpisywaniem CLA? bo niby przechodzi, ale pokazuje jak nie zaakceptowane

nie myśleliście czasem, żeby pozwolić na squash merge? bardzo życie ułatwia jak się robi wiele commitów małych zmian, wiem, że mogę to obejść, ale robienie squash rebase i force push za każdym razem jest ryzykowne.

[Robert Błaszczak]

Wow, zapowiada się bardzo ciekawie...

[Zibi_007]

[klew]

Squash mogę zrobić przy merge'u.
Co do długości linii - ja używam jakichś automatycznych narzędzi, które formatują kod.

PR do cloud oraz core daj do brancha develop. Na device jest ok, bo ja tam tylko na main pracuję.

[[email protected]]

Squash mogę zrobić przy merge'u.
Co do długości linii - ja używam jakichś automatycznych narzędzi, które formatują kod.

PR do cloud oraz core daj do brancha develop. Na device jest ok, bo ja tam tylko na main pracuję.

Dałem do develop?

[klew]

Dałem do develop?

Hm, tak. Wydaje mi się że wczoraj widziałem do mastera...

[[email protected]]

Dałem do develop?

Hm, tak. Wydaje mi się że wczoraj widziałem do mastera...

Owszem, wczoraj

[SOYER]

Brawo Rafał!!! Super:)

[pzygmunt]

Brawo