Skip to content

fix crash when removing node from group that contains note.#12551

Merged
mjkkirschner merged 1 commit intoDynamoDS:masterfrom
mjkkirschner:dyn4505
Jan 19, 2022
Merged

fix crash when removing node from group that contains note.#12551
mjkkirschner merged 1 commit intoDynamoDS:masterfrom
mjkkirschner:dyn4505

Conversation

@mjkkirschner
Copy link
Member

@mjkkirschner mjkkirschner commented Jan 18, 2022

Purpose

https://jira.autodesk.com/browse/DYN-4505

add null check to linq query
add test

Declarations

Check these if you believe they are true

  • The codebase 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
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated

Release Notes

fix crash when removing node from group that contains note.

add test
@QilongTang QilongTang added this to the 1.3.1 milestone Jan 18, 2022
@QilongTang
Copy link
Contributor

LGTM.. And you are quick :)

@mjkkirschner mjkkirschner modified the milestones: 1.3.1, 2.13.1 Jan 18, 2022
@mjkkirschner
Copy link
Member Author

@mjkkirschner mjkkirschner merged commit c4c8e57 into DynamoDS:master Jan 19, 2022
mjkkirschner added a commit to mjkkirschner/Dynamo that referenced this pull request Jan 19, 2022
add test

Co-authored-by: michael kirschner <[email protected]>
mjkkirschner added a commit that referenced this pull request Jan 19, 2022
add test

Co-authored-by: michael kirschner <[email protected]>

Co-authored-by: michael kirschner <[email protected]>
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