[DYN-7833] Run All should be disabled in Custom Nodes#86
Conversation
|
Thank you @ivaylo-matov I am curious with this code, if user is simply switching back and forth between homeworkspace and custome node workspace, would the execution time data come back after switching from custom node workspace? If not, do we need to cache it after switching? |
|
@QilongTang , right now it does not. |
|
@ivaylo-matov Switching between workspaces is a pretty normal workflow in Dynamo, I would encourage you to implement caching. The way I see it is that until homeworkspace is closed/ cleared, we dont need to totally wipe out the data when opening custom node workspace, but caching it so we can still reuse when switched back, what do you think? |
reddyashish
left a comment
There was a problem hiding this comment.
LGTM. Will merge once the checks pass.
PR aims to address DYN-7833 and the conversation in this Slack tread
The "Run All" button is now disabled when the workspace is in Custom Node mode. The button is greyed out and a tooltip is displayed to explain that the button is not available in this mode.
@QilongTang
@reddyashish
@dnenov