Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring and update txt support #41

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sokarovski
Copy link

Hey there, I needed support for txt record update in the service so while i did that i also did a little bit of refactoring. Hope you like it :D

@itsmepetrov
Copy link

Hi @watson, please review this PR.
We really need this functionality because we want to get rid of binding in HAP-NodeJS.

I hope it doesn't take much time.

@itsmepetrov
Copy link

@watson please don't ignore this request, if you don't have enough time to support this library you can give access to @sokarovski, because, as I understand, he already know code base of the library and can accept and review another PRs from the community.

@sprut666666
Copy link

+1

18 similar comments
@earthwood
Copy link

+1

@IevgeniiSkorobogatov
Copy link

+1

@permadm
Copy link

permadm commented Mar 16, 2018

+1

@Urbanboom
Copy link

+1

@permadm
Copy link

permadm commented Mar 18, 2018

+1

@Dedcad
Copy link

Dedcad commented Mar 18, 2018

+1

@stalkerj
Copy link

+1

@iliya2003
Copy link

+1

@Ponf
Copy link

Ponf commented Mar 19, 2018

+1

@Djzabrail
Copy link

+1

@LyoshaDovolen
Copy link

+1

@440777
Copy link

440777 commented Mar 19, 2018

+1

@Andrey909
Copy link

+1

@Nebr88
Copy link

Nebr88 commented Mar 19, 2018

+1

@Anzic23
Copy link

Anzic23 commented Mar 19, 2018

+1

@KarpushinVA
Copy link

+1

@Dedcad
Copy link

Dedcad commented Mar 19, 2018

+1

@custman
Copy link

custman commented Mar 19, 2018

+1

@Maxmudjon
Copy link

+1

2 similar comments
@follesoe
Copy link

+1

@SerebryanskiySergei
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.