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

feat: Dynamic hooks for END_PRINT and START_PRINT macros #669

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mjonuschat
Copy link

@mjonuschat mjonuschat commented Oct 29, 2024

Implement a variation of the startprint_actions for end_print that looks for custom actions/modules that start with _END_PRINT_ACTION_ for the END_PRINT macro and modernize the START_PRINT macro to utilize the same approach for modules prefixed with _START_PRINT_ACTION.

The legacy _MODULE_CUSTOM1 etc modules have been left in place to not break existing installations.

@mjonuschat mjonuschat force-pushed the feature/custom-end-print-hooks branch from a1cec62 to fb4a502 Compare December 16, 2024 05:52
@mjonuschat mjonuschat changed the title feat: Custom modules for END_PRINT macro feat: Dynamic hooks for END_PRINT and START_PRINT macros Dec 16, 2024
@mjonuschat mjonuschat force-pushed the feature/custom-end-print-hooks branch 2 times, most recently from 2cd394d to 32b6b7b Compare December 20, 2024 05:27
Implement a similar approach to START_PRINT to enable users to hook
into the END_PRINT macro. This approach is more dynamic in that it
allowes to call any Gcode Macro that starts with `_END_PRINT_ACTION_`
and verifies that the desired action exists before calling it.
Revamp the current logic to hook into the START_PRINT macro and
implement the same dynamic approach as END_PRINT for Gcode macros
starting with `_START_PRINT_ACTION`, again verifying existence
before calling the macro.
@mjonuschat mjonuschat force-pushed the feature/custom-end-print-hooks branch from 32b6b7b to d25823b Compare December 21, 2024 04:02
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.

1 participant