Skip to content

Add comments to Device class#4

Open
Pr0mises wants to merge 5 commits intoanszom:masterfrom
Pr0mises:comments
Open

Add comments to Device class#4
Pr0mises wants to merge 5 commits intoanszom:masterfrom
Pr0mises:comments

Conversation

@Pr0mises
Copy link
Contributor

@Pr0mises Pr0mises commented Jul 22, 2024

This adds comments to base.js and RAC_056905_WW.js

I wanted to do multiple PRs for the ease of allowing and declining certain changes

Pr0mises added 5 commits July 22, 2024 20:21
Fixes the off behaviour for home assistant using the off mode
I added a combined swing mode which also adds the possibility to change the horizontal swing mode as well vertical
@anszom
Copy link
Owner

anszom commented Jul 23, 2024

Thanks! I'm a bit busy right now but I'll take a look in a few days

@anszom
Copy link
Owner

anszom commented Jul 31, 2024

I like the comments, but I have two issues with the PR.
First, please do not reformat the code. Tab indentation may not be the most common nowadays, but I'm going to stick with it :)
Second, I'm confused about the reverts. My guess is that you wanted to keep each change separate, not undo your changes...

@Pr0mises
Copy link
Contributor Author

Ah sorry about that, I missed the spaces change. Looks like VScode did an auto format.

Yes I wanted to make them separate and used the wrong branch. Wasn't sure if it's possible like that.

If not I'll make the changes next week

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.

2 participants