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

backup_feedrate_percentage no longer requires PRUSA_MMU2 #20345

Closed
wants to merge 1 commit into from

Conversation

nb-rapidia
Copy link
Contributor

Description

This is a pull request for issue #20333.

The feedrate backup and restore feature for M220 is mentioned in the documentation, but what is not mentioned is that it is gated behind the PRUSA_MMU2 macro.

Benefits

This PR amends the code to match the documentation by enabling the backup and restore feature regardless of PRUSA_MMU2.

Configurations

No special configuration required.

Related Issues

#20333 by @thomaslai.

This handy feature is mentioned in the documentation, but not that it requires PRUSA_MMU2. This commit amends the code to match the documentation.
@thisiskeithb
Copy link
Member

Thanks for your contribution! Unfortunately we can't accept PRs directed at release branches. We make patches to the bugfix branches and only later do we push them out as releases. Please redo this PR starting with the bugfix-2.0.x branch and be careful to target bugfix-2.0.x when resubmitting the PR.

It may help to set your fork's default branch to bugfix-2.0.x.

See Marlin's Contributing Code with Pull Requests guide for full instructions.

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