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

Invert sense #39

Closed
wants to merge 3 commits into from
Closed

Invert sense #39

wants to merge 3 commits into from

Conversation

agreen
Copy link
Contributor

@agreen agreen commented Jul 1, 2017

Closes #36

@agreen
Copy link
Contributor Author

agreen commented Jul 15, 2017

Just checking in on this, if you have any questions/issues.

@kantlivelong
Copy link
Owner

Haven't had a chance to test this yet. One thing I noticed that could be changed is the senseGPIOPinPUD drop down in the settings UI. I think it would be nicer to put it horizontally before or after the invert checkbox with no label. Change "Off" to "No Pull Up/Down" or something similar.

Thoughts?

@agreen
Copy link
Contributor Author

agreen commented Jul 17, 2017

I was concerned with the length of the drop down. I originally had it something along those lines. I figured the page was better suited to vertical than horizontal, but I'm up for anything. Let me know if you want me to modify the PR.

@kantlivelong
Copy link
Owner

As long as it sits in one line it should be fine.

Thanks for the help!

@kantlivelong
Copy link
Owner

Alright I've finally tested the code changes. Looks good! 👍

If you prefer I can make the UI changes. Let me know.

@agreen
Copy link
Contributor Author

agreen commented Jul 20, 2017

Might be faster if you do, I've run into a few setbacks on my side. Glad you like it, and thank you for all the work you've put into the plugin.

@kantlivelong
Copy link
Owner

Alright. Added to my TODO list.

Thanks for your additions!

@kantlivelong
Copy link
Owner

Merged

kantlivelong added a commit that referenced this pull request Sep 17, 2017
Add getPSUState API Command (#34)
Set initial state of the switching GPIO accordingly (#40)
Add Pseudo On/Off G-Code Switching (#26)
Added M106 to autoOnTriggerGCodeCommands defaults (#45)
Move static power off dialog from jinja2 template to js (#46)
Change power off dialog message
Add invert and pull-up/down options for Sensing (#39)
Add support for sensing via system commands
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants