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.

Sign up for GitHub

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

Jump to bottom

Add AC's Carrier 42QG5A580SC and AUX YKR-H/006E #3284

Merged
merged 7 commits into from
Dec 15, 2023
Merged

Add AC's Carrier 42QG5A580SC and AUX YKR-H/006E #3284

merged 7 commits into from
Dec 15, 2023

Conversation

gusdleon
Copy link
Contributor

@gusdleon gusdleon commented Dec 11, 2023 •
edited by skotopes
Loading

What's new

  • Added the codes for the AC's:
    • the old Carrier expression (remote 42QG5A580SC)
    • AUX (remote YKR-H/006E)

Verification

  • Followed the instructions and after testing the new created control, all buttons work as intended.

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@hedger hedger added the Infrared Infrared app label Dec 12, 2023
@skotopes skotopes added the New Feature Contains an IMPLEMENTATION of a new feature label Dec 12, 2023
skotopes
skotopes previously approved these changes Dec 12, 2023
View reviewed changes
@gusdleon gusdleon changed the title Add Carrier 42QG5A580SC Add AC's Carrier 42QG5A580SC and AUX YKR-H/006E Dec 12, 2023
@skotopes skotopes merged commit 11ac624 into flipperdevices:dev Dec 15, 2023
9 checks passed
haisenteck added a commit to haisenteck/flipperzero-Haisenteck that referenced this pull request Dec 19, 2023
@haisenteck @skotopes
Add AC's Carrier 42QG5A580SC and AUX YKR-H/006E (flipperdevices#3284)
e7169b8 
* Add Carrier 42QG5A580SC
* Add AC Model: AUX YKR-H/006E
* Cleanup ac.ir
* Infrared: Move Carrier 42QHB12D8S to the back of universal remote
* Infreared: more universal ac cleanup

Co-authored-by: あく <alleteam@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skotopes skotopes skotopes approved these changes

@DrZlo13 DrZlo13 Awaiting requested review from DrZlo13 DrZlo13 is a code owner

@hedger hedger Awaiting requested review from hedger hedger is a code owner

@gsurkov gsurkov Awaiting requested review from gsurkov gsurkov is a code owner

Assignees
No one assigned
Labels
Infrared Infrared app New Feature Contains an IMPLEMENTATION of a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@gusdleon @skotopes @hedger