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

[Part Design] allow for preselecting loft sections and pipe spines pr… #3757

Closed
wants to merge 2 commits into from

Conversation

mwganson
Copy link
Contributor

…ior to invoking command

Thank you for creating a pull request to contribute to FreeCAD! To ease integration, please confirm the following:

  • Branch rebased on latest master git pull --rebase upstream master
  • Unit tests confirmed to pass by running ./bin/FreeCAD --run-test 0
  • Commit message is well-written
  • Commit message includes issue #<id> or fixes #<id> where <id> is the associated MantisBT issue id if one exists

And please remember to update the Wiki with the features added or changed once this PR is merged.
Note: If you don't have wiki access, then please mention your contribution on the 0.19 Changelog Forum Thread.


@mwganson
Copy link
Contributor Author

@mwganson
Copy link
Contributor Author

I've discovered a bug after additional testing. Preview during the dialog isn't working as expected for subtractive pipes and after completion the previous tip is still visible.

@mwganson
Copy link
Contributor Author

I think I have all the bugs fixed now.

@mwganson
Copy link
Contributor Author

I tried looking through the travis build failure, but I don't see why it failed or even that it failed, much less how to fix.

@Jee-Bee
Copy link

Jee-Bee commented Sep 10, 2020

good to see that there is some work regarding selection methods in PD!!

@donovaly
Copy link
Member

Can you please merge master to this PR because I get a merge conflict?

@wwmayer
Copy link
Contributor

wwmayer commented Sep 15, 2020

Merged.

@wwmayer wwmayer closed this Sep 15, 2020
@mwganson mwganson deleted the pd_loft branch September 4, 2021 17:52
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.

None yet

4 participants