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

Call Create Group #343

Merged

Conversation

kThorsager
Copy link
Contributor

Issues addressed by this PR

Closes #339
Calls the Create Group and adds some error checking

Test files

https://burohappold.sharepoint.com/:f:/s/BHoM/EkRp-GxKlr1FkxpK7WtOMDkB95ttAkzAfw5WtvL-ZC-m4w?e=55xaF2

Changelog

Additional comments

@kThorsager kThorsager added the type:feature New capability or enhancement label Mar 31, 2020
@kThorsager kThorsager added this to the BHoM 3.2 β MVP milestone Mar 31, 2020
@kThorsager kThorsager self-assigned this Mar 31, 2020

if (objects.First() is IBHoMGroup)
{
success = CreateCollection(objects as dynamic);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No Idea how to get this to work otherwise

Copy link
Contributor

@IsakNaslundBh IsakNaslundBh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working for Bars and Nodes. To get this running for panels as well, I would add in:

image

Before you do, that file has been changed in another PR merged, so would update from master before changing it.

@kThorsager kThorsager requested a review from IsakNaslundBh April 9, 2020 12:52
Copy link
Contributor

@IsakNaslundBh IsakNaslundBh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine

@IsakNaslundBh IsakNaslundBh merged commit c1002d4 into master Apr 9, 2020
@IsakNaslundBh IsakNaslundBh deleted the Robot_Toolkit-#339-Create-Group-is-never-called branch April 9, 2020 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New capability or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Robot_Toolkit: Create Group is never called
2 participants