Skip to content

override the local revit model when saving #3312

Merged
pinzart90 merged 1 commit intoD4DA_2026from
fix_overrite_issue
Apr 10, 2026
Merged

override the local revit model when saving #3312
pinzart90 merged 1 commit intoD4DA_2026from
fix_overrite_issue

Conversation

@pinzart90
Copy link
Copy Markdown
Collaborator

Bogdan: When I run a graph that modifies the rvt it looks like we don't get back the modified version ? Is this expected ?

Purpose

(FILL ME IN) This section describes why this PR is here. Usually it would include a reference
to the tracking task that it is part or all of the solution for.

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

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

@pinzart90 pinzart90 changed the title Update DAEntrypoint.cs override the local revit model when saving Apr 9, 2026
Copy link
Copy Markdown

@awigginsadsk awigginsadsk left a comment

Choose a reason for hiding this comment

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

LGTM

@pinzart90 pinzart90 merged commit 4364aa9 into D4DA_2026 Apr 10, 2026
9 checks passed
@pinzart90 pinzart90 deleted the fix_overrite_issue branch April 10, 2026 08:24
pinzart90 added a commit that referenced this pull request Apr 16, 2026
Bogdan: When I run a graph that modifies the rvt it looks like we don't
get back the modified version ? Is this expected ?

(FILL ME IN) This section describes why this PR is here. Usually it
would include a reference
to the tracking task that it is part or all of the solution for.

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](https://github.com/DynamoDS/Dynamo/wiki/Coding-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.

(FILL ME IN) Reviewer 1  (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

(FILL ME IN, Optional) Names of anyone else you wish to be notified of
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