-
Notifications
You must be signed in to change notification settings - Fork 388
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
Add r718pa,r718pa4,r718pb,rp02,r718n3 support #718
Add r718pa,r718pa4,r718pb,rp02,r718n3 support #718
Conversation
hi @netvox-tech this PR is the same as #717 which one is the one to review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add images for the devices as well?
Dear Sir,
please review #718. Thank you.
On Thu, 07 Dec 2023 04:31:35 -0800, Jaime Trinidad wrote
… hi @netvox-tech this PR is the same as #717 which one is the one to review?
--
Reply to this email directly or view it on GitHub:
#718 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
--
Regards......your name (http://www.netvox.com.tw)
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
|
@netvox-tech please add images of the devices, when we merge it the information will be deployed https://www.thethingsnetwork.org/device-repository/ without background please |
@netvox-tech any update? |
@netvox-tech any update here? |
You said it was approved, then you mentioned missing images, and finally you mentioned conflicts. So what was the final situation? |
The situation is that it's missing images. Please add those and then it should be good. |
@netvox-tech Could you please still add the images so that we can approve of this PR? |
@netvox-tech I had not expected you to make a seperate PR for the images. I see all the files from this PR are in the DR now, so can we close this one? |
From what i can see these files are added in the repo with the other PRs, closing this. |
Add r718pa,r718pa4,r718pb,rp02,r718n3 support