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

3dview wrapper 2018 #1999

Merged
merged 2 commits into from
Aug 13, 2018
Merged

Conversation

ksobon
Copy link
Contributor

@ksobon ksobon commented Apr 11, 2018

Purpose

This addresses this issue:

#1636
#2002

It also addresses the issue of plain old skipping 3d view templates from any selection and wrapper routines.

Declarations

Check these if you believe they are true

  • The code base is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • Snapshot of UI changes, if any.

Reviewers

@mjkkirschner

FYIs

This used to be a PR but i messed up so here it is again. I ran tests for all relevant classes:

image

image

@ksobon ksobon changed the base branch from Revit2019 to Revit2018 April 11, 2018 14:27
@ZiyunShang ZiyunShang merged commit b132501 into DynamoDS:Revit2018 Aug 13, 2018
@ksobon ksobon deleted the 3dview-wrapper-2018 branch August 13, 2018 12:39
AndyDu1985 added a commit that referenced this pull request Aug 14, 2018
Merge pull request #1999 from ksobon/3dview-wrapper-2018
ZiyunShang added a commit to ZiyunShang/DynamoRevit that referenced this pull request Dec 10, 2018
ZiyunShang added a commit to ZiyunShang/DynamoRevit that referenced this pull request Dec 10, 2018
ZiyunShang added a commit to ZiyunShang/DynamoRevit that referenced this pull request Dec 10, 2018
AndyDu1985 pushed a commit that referenced this pull request Dec 11, 2018
* Change RevitAPI path to match D4R CICD

* update version to 27 resolve CICD break

* resolve Nugets NET47 updates

* Revert "Merge pull request #1999 from ksobon/3dview-wrapper-2018"

This reverts commit 535c8ba.

* Delete DynamoPlayer from D4R CICD
ZiyunShang added a commit that referenced this pull request Dec 14, 2018
* Change RevitAPI path to match D4R CICD

* update version to 27 resolve CICD break

* resolve Nugets NET47 updates

* Revert "Merge pull request #1999 from ksobon/3dview-wrapper-2018"

This reverts commit 535c8ba.

* Delete DynamoPlayer from D4R CICD

* update version to 30
AndyDu1985 pushed a commit that referenced this pull request Dec 18, 2018
* Change RevitAPI path to match D4R CICD

* update version to 27 resolve CICD break

* resolve Nugets NET47 updates

* Revert "Merge pull request #1999 from ksobon/3dview-wrapper-2018"

This reverts commit 535c8ba.

* Delete DynamoPlayer from D4R CICD

* update version to 30

* Set 5 tests to ignore because of DynamoRaaS removed

* Add Change Log to track each change in version upgrade
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