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

A new ID for Mini 3 #12

Closed
wants to merge 1 commit into from
Closed

Conversation

KaloNikolov
Copy link

I just bought a new RM Mini 3 (firmware ver. 40) and it reports the ID in the above code change.
I tested both code learning and emitting and it works just fine after the local edit.
I see there is another active pull request which states it is for Mini 3, but the IDs are different

I just bought a new RM Mini 3 (firmware ver. 40) and it reports the ID in the above code change. 
I tested both code learning and emitting and it works just fine after the local edit.
@fbacker
Copy link

fbacker commented Apr 27, 2019

This project never updated so I forked it with you fix and another devices as well
https://github.com/fbacker/broadlinkjs-rm

@lprhodes
Copy link
Owner

@KaloNikolov Thanks for providing the device ID, I've included it within the latest version

@lprhodes lprhodes closed this Dec 31, 2019
401Unauthorized pushed a commit to 401Unauthorized/broadlinkjs-rm that referenced this pull request Mar 9, 2021
* Updated issues link
* Adds 6539 support
* Fixed packet MAC/ID mapping in headers
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.

None yet

3 participants