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

Formatted Codes-for-IR-Remotes consistently as tables #1288

Merged
merged 3 commits into from
Nov 11, 2023

Conversation

robertoostenveld
Copy link
Contributor

The page https://tasmota.github.io/docs/Codes-for-IR-Remotes/ had very inconsistent formatting for the different entries.

In this PR I have formatted all lists and tables consistently and made some small cleanups to the headings and to the comments in each section. I have not changed any of the technical content.

@robertoostenveld
Copy link
Contributor Author

Hmm, I don't understand why there are conflicts. I thought my PR was a clean change relative to master. I will investigate...

Make heading levels consistent.

Added IR codes for Goodmans, Fishtec, and Edifier, which I missed because my initial PR was based on the development branch but these were added to the master branch.
@robertoostenveld
Copy link
Contributor Author

Hmm, I don't understand why there are conflicts.

It turned out that I started off from the development branch. I rebased my changes on the master branch and added the three new devices that I missed.

It is now a clean PR relative to master.

@DeflateAwning
Copy link
Contributor

This would be a really good one to get merged. That page is a bit of a mess currently

@robertoostenveld
Copy link
Contributor Author

Once it is merged, I also plan to add IR remote codes for my Samsung TV and NAD D3020 audio amplifier.

@mendel5
Copy link
Contributor

mendel5 commented Nov 11, 2023

I think this is a useful change, @robertoostenveld , thank you!

There is a merge conflict due to this change:

Could you maybe update your pull request to include this new change?

@robertoostenveld
Copy link
Contributor Author

Sure, done!

@barbudor barbudor merged commit ea09f23 into tasmota:master Nov 11, 2023
1 check passed
@mendel5
Copy link
Contributor

mendel5 commented Nov 11, 2023

Thank you!

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.

4 participants