mdawsonuk / streamdeck-agh Goto Github PK

Currently button text doesn't update if there is an auth error or if the connection times out. This should be replaced with text saying "Incorrect credentials" or "Connection timed out"

Currently all Actions use the same generic AGH logo - instead these should be vector images which are individually distinguishable and identify clearly the kind of Action the button is.

Elgato have a page detailing the styling guidelines for Stream Deck plugins. This should be conformed to when creating the images/icons.

Use the showAlert event to show errors on the Stream Deck rather than using alert();

Ensure code is well documented throughout before v1.0.0.

• Explain in event callbacks where they are called from ie didReceiveGlobalSettings
• Docstrings for functions and classes

AGH needs an API wrapper rather than making the API calls directly in code.js.

Rather than having per-button settings, the Authentication settings should probably be global. While it is possible that a user might have multiple AGH instances per-network, it is unlikely they will need to control multiple AGH instances per-Stream deck. Therefore, it makes sense for the Authentication settings to be stored globally.

Hi,

The plugin isn't working for me anymore. I've added it to the Stream Deck and when I press the test button I get the 'Success!' message. But I then want to toggle the AdGuard protection, nothing happens. In the AdGuard logs I can see this pop up

POST 192.168.1.127:83 /control/dns_config: only content-type application/json is allowed

Might have something to do with this? AdguardTeam/AdGuardHome#4970

Submit the AdGuard Home plugin to the Elgato Stream Deck Plugin Store

Updated README to talk about how to set up a development environment, getting started, brief about section detailing functionality.

Add a dropdown to the property inspector which allows users to select a polling period.
Currently thinking of having the options be 15s, 30s, 60s, 120s and 150s.

Building on #1, pull the specified stat and overlay it on the button. This would benefit from #18 being completed first.