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

[TD]add fallback default template #13784

Merged
merged 1 commit into from May 3, 2024

Conversation

WandererFan
Copy link
Contributor

This PR implements a fix for issue issue #13539. In PR #13368 the default template was renamed. This PR adds a new fallback template and adjusts the preference function to return the new default template file,

Additional discussion on forum: https://forum.freecad.org/viewtopic.php?t=87239&sid=49290882ca3ba68ee813770b981bb5ce

@github-actions github-actions bot added the WB TechDraw Related to the TechDraw Workbench label May 2, 2024
@PaddleStroke
Copy link
Contributor

Ah you already made a fix here then. So #13794 is duplicate.

However in your PR you are adding an empty file to be the new default? Or is it a visibility issue on github? But the file looks like an empty rectangle.

@maxwxyz
Copy link
Collaborator

maxwxyz commented May 2, 2024

I guess to provide a blank page

@WandererFan
Copy link
Contributor Author

It is a blank A4 sheet. Seemed like a reasonable fall back.

@WandererFan WandererFan merged commit af2b8b6 into FreeCAD:main May 3, 2024
10 checks passed
@WandererFan WandererFan deleted the defaultTemplate branch May 3, 2024 02:15
@PaddleStroke
Copy link
Contributor

PaddleStroke commented May 3, 2024 via email

@FEA-eng
Copy link
Contributor

FEA-eng commented May 5, 2024

Right, the error is gone now but a blank page is created by default. Can't it just be the same default template as before?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WB TechDraw Related to the TechDraw Workbench
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants