Dave T

Results 103 comments of Dave T

Sorry that there are problems causing difficulty here. If there are flaky/failing CI tests on the dev branch then probably you are not the only one to notice. Rebase and...

It looks like you have rebased and force pushed. I can't easily see what has changed since I last reviewed and I would prefer not to re-read all >1000 lines....

Ok, this is quite a big job now. It's close to a rewrite and using quite a new python library (so maybe that needs reviewing as well). There is a...

+1 from me. QR codes and RFID tag URLs could then be used to trigger automations without apps.

@esev what type of CSRF protection did you have in mind? I don't think any of the use cases wanting this can set header tokens.

Unauthenticated webhooks that do things are a security hole. But sometimes necessary. I suggest: - Off by default - Red warning text in docs next to turning them on, with...

> Great ideas here. All being ignored though. There are currently 370 open PRs... maybe you could help by reviewing the PRs that @esev has kindly created? If not the...

@byroncoetsee @FutureTense I don't think anyone is objecting to this feature. It's just not been reviewed yet. You could help by reviewing the PRs - even a partial review helps....

@zsquareplusc Please could you advise whether this is still actively being maintained? I see no commits for ~2y but definitely people are still using this.

Update: I'm slightly less happy with this than I was. Since creating the PR I've realised that the template repo is probably intended for a collection of add-ons, not just...