python-adguardhome icon indicating copy to clipboard operation
python-adguardhome copied to clipboard

Feature: Client Resources

Open offbyone opened this issue 2 years ago • 27 comments

  • added getters for the list of client / autoclient / supported tags resources

Proposed Changes

Adding support for the Client / AutoClient resources in AdGuard Home

The first commit here is to open discussion on structure and naming; if this is desirable in this state, I will add the update operations to it as well.

I don't intend to get super clever with remote ORM stuff here; I only want to have simple "add", "update", "delete" methods for them that refer to the APIs directly.

Related Issues

(Github link to related issues or pull requests)

offbyone avatar Jan 15 '23 07:01 offbyone

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Jan 15 '23 16:01 sonarqubecloud[bot]

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Feb 15 '23 08:02 github-actions[bot]

Ping?

offbyone avatar Feb 15 '23 14:02 offbyone

I'll ping too. Please test/merge this. I'd like to use it upstream with HomeAssistant

bendalton avatar Mar 13 '23 18:03 bendalton

@bendalton and @offbyone

I hate to break it to you but, Frenck mentioned in a HomeAssistant PR, that he doesn't expect to do maintenance on this library in the short-term.

See: https://github.com/home-assistant/core/pull/83505#issuecomment-1342296729 for more information.

sandervankasteel avatar Mar 15 '23 16:03 sandervankasteel

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Mar 15 '23 16:03 sonarqubecloud[bot]

I'll ping too. Please test/merge this. I'd like to use it upstream with HomeAssistant

I hate to bring it to you, but if you goal is to use this in Home Assistant: You can't.

To make this possible, we need an identifier. A request for this is open @ the AdGuard Home team.

So, if that is your only goal, I suggest closing this PR as it has no foreseeing future.

frenck avatar Mar 15 '23 16:03 frenck

Nope, I'm using this for scripting independent of HA.

offbyone avatar Mar 15 '23 19:03 offbyone

Sorry I now realize I responded to a non-author, sorry about that 👍

../Frenck

frenck avatar Mar 15 '23 19:03 frenck

Okay; I'll add update methods here as well and then update the PR. Thanks.

offbyone avatar Mar 15 '23 20:03 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Apr 15 '23 08:04 github-actions[bot]

I'm still interested in merging this, just haven't had the time to finish it up.

offbyone avatar Apr 16 '23 16:04 offbyone

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Apr 18 '23 05:04 sonarqubecloud[bot]

@frenck I'm not able to label it in a way that'll let the PR labels workflow pass; could you do that?

offbyone avatar Apr 25 '23 22:04 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar May 26 '23 08:05 github-actions[bot]

I am still interested in merging this, I just need the maintainer to okay it.

offbyone avatar May 26 '23 16:05 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Jun 26 '23 08:06 github-actions[bot]

I am still interested in merging this, and waiting for @frenck to label it and approve it.

offbyone avatar Jun 26 '23 13:06 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Jul 27 '23 08:07 github-actions[bot]

Yup, still interested. @frenck?

offbyone avatar Jul 27 '23 14:07 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Aug 28 '23 08:08 github-actions[bot]

Yup, still interested. @frenck are you maintaining this library any more? If not, can I have the PyPi keys and I'll start publishing it?

offbyone avatar Aug 29 '23 18:08 offbyone

I still have plans, right now, I am traveling (for a few weeks to come still).

frenck avatar Aug 29 '23 19:08 frenck

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Sep 29 '23 08:09 github-actions[bot]

ping

offbyone avatar Sep 29 '23 14:09 offbyone

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Oct 30 '23 08:10 github-actions[bot]

I would still like to see this merge.

offbyone avatar Oct 30 '23 15:10 offbyone