Copilot

Results 4915 comments of Copilot

> @copilot since usermod names are known by the controller, if a pin is owned by a UM, use the UM name string and omit the UM list in JS....

> @copilot now you made it worse: the unknown pins are unuseable so omit those, you omitted all available pins. also touch button still does not show the state like...

> @copilot you are now reading touch states correctly but touch buttons still do not show the state in the UI Fixed in 1e77bc0. Touch button state now shows in...

> @copilot touch pins still do not show ANY state dot. please rethink what could be the issue. Fixed in 113cdf3. Added a fallback for button-owned pins (`PinOwner::Button`) that aren't...

> @copilot you are still using the UM strings in the UI which is "hardcoded" and not future proof. each usermod has "name[]" string, use that instead. also consolidate JS...

> @copilot move the UI code to a seperate htm file named settingspins, add a button in the settings.htm below the LED settings and remove it from the info tab....

> @copilot instead of doing a re-implementation of code in button.cpp, modify button.cpp with helper functions such that you can reuse functions to detect a shortPressAction and longPressAction, I do...

> @copilot rebase your work to the latest changes in main. also use the existing /settings/s.js?p=2 which already contains some pin info and extend it to contain touch capability. Done...

> @copilot I updated from main, please fix for new button handling Fixed in 00aadf2. Updated `serializePins()` to use the new `std::vector buttons` API: - Access button pin via `buttons[b].pin`...

> @copilot on ESP8266, GPIO17 is not defined, only 0-16 exist. also on any of the ESP32 variants, ADC2 channels can not be used as its occupied by wifi. >...