-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Cura 12138 export for support option #19644
base: main
Are you sure you want to change the base?
Conversation
@@ -33,7 +35,7 @@ def setExportModel(self, model: SettingsExportModel) -> None: | |||
if self._ucp_model != model: | |||
self._ucp_model = model | |||
|
|||
def _write(self, stream, nodes, mode=WorkspaceWriter.OutputMode.BinaryMode): | |||
def _write(self, stream, nodes, mode, include_log): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Include log is a bool trap. I'd also argue that instead of removing the default for the mode, we should just set the include_log to be False by default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right, I have removed the mode
default value not because of the include_log
, but because it is a "private" method and only called at one place, with a value for the mode
. Restoring the default value and setting include_log
to False
is also fine if you prefer.
CURA-12138
Requires Ultimaker/Uranium#970
"exported-(package/project)-for-support-timestamp/random-number).3mf"
ex: exported-project-for-support-20240911-13-13.3mf