-
-
Notifications
You must be signed in to change notification settings - Fork 19.3k
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
Add assign PIN to G-CODE macro feature #18389
Add assign PIN to G-CODE macro feature #18389
Conversation
I don't know why ANYONE who has authority does not accept this method. |
Deferred until after the next release. |
What is the state of this? We've certainly had our next release. @MoellerDi do you still consider these changes to be Draft? |
@sjasonsmith: I'll work on it this weekend to get it to the latest code base and do some cleanup. Should be ready for testing by Monday. @android444: as it was your idea; anything from your end to be added or are you ok to proceed with this PR? Are you ok to get this merged (once ready) or do you prefer to further work on your PR #17586 ? |
…ed; USER_GCODE_PIN_n will always be initiated
6832931
to
3652f24
Compare
IMHO this PR is now ready for review and testing |
looking forward to test it, :) Is it possible to add different scenarios for a single button, to override the limitation of available pins : long push 3 sec |
4274255
to
a97a1ae
Compare
I'll have a look at that and see what I can do once we are sure the basic functionality works and got tested ;) |
…ugfix-2.0.x-PR-assign_PIN_to_G-CODE_macro_feature
Co-Authored-By: android444 <24375898+android444@users.noreply.github.com>
Hi, @MoellerDi Where is the mistake? |
himm, that #if looks wrong. which means it appends the _PIN to the name, so it should be #if PIN_EXISTS(BUTTON1) |
Hi @ellensp Should I fix all BUTTON conditions? |
yes, are you going to create PR? I will If you don't. |
...unfortunately I can't create PR. |
ok ill do it. |
PR to fix this has been created |
Co-Authored-By: android444 <24375898+android444@users.noreply.github.com> Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-Authored-By: android444 <24375898+android444@users.noreply.github.com> Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-Authored-By: android444 <24375898+android444@users.noreply.github.com> Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Co-Authored-By: android444 <24375898+android444@users.noreply.github.com> Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Description
This PR is based on the great idea and work from @android444 (PR #17586) and based on the communication between him and @thinkyhead.
This PR might not be ready and should be seen as proof of concept.
(it was tested on my own printer and works fine)
Benefits
This PR once completed is enabling the user to assign a PIN to predefined g-code macros
by using already existing custom menu option.Related Issues
no issues found but it's related to PR #17586
maybe related to #7932