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

DYN-6554 Revert UI Blocking Function calls #14766

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Conversation

QilongTang
Copy link
Contributor

Purpose

Per https://jira.autodesk.com/browse/DYN-6554, This PR reverts a part of the changes from #13973 because it causes some undesired UI side effects to the preview bubble when launching the preferences panel and package manager. Morpheus team decided to keep this function call only for guided tour and borrow the overlay mechanism from the guided tour to apply to Preferences Panel and PM open by another PR.

Before:
image

After:
image

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
  • This PR contains no files larger than 50 MB

Release Notes

N/A

Reviewers

@RobertGlobant20

FYIs

@DynamoDS/dynamo

@QilongTang QilongTang added this to the 3.0 milestone Dec 15, 2023
Copy link

UI Smoke Tests

Test: success. 2 passed, 0 failed.
TestComplete Test Result
Workflow Run: UI Smoke Tests
Check: UI Smoke Tests - net8.0

Copy link

UI Smoke Tests

Test: success. 2 passed, 0 failed.
TestComplete Test Result
Workflow Run: UI Smoke Tests
Check: UI Smoke Tests - net6.0

Copy link
Contributor

@RobertGlobant20 RobertGlobant20 left a comment

Choose a reason for hiding this comment

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

LGTM

@QilongTang
Copy link
Contributor Author

QilongTang commented Dec 15, 2023

I dont understand.. PR check on master-5 finished without any regression but indicate fail state so the PR check here never finished. See https://master-5.jenkins.autodesk.com/job/Dynamo/job/DynamoSelfServe/job/pullRequestValidation/12587/

At the same time, self serve on master-15 passed at https://master-15.jenkins.autodesk.com/view/DYN/job/DYN-DevCI_Self_Service/1355/

Merging..

@QilongTang QilongTang merged commit 3f18208 into master Dec 15, 2023
21 of 22 checks passed
@QilongTang QilongTang deleted the UIBlockingOperation branch December 15, 2023 16:28
QilongTang added a commit that referenced this pull request Dec 15, 2023
QilongTang added a commit that referenced this pull request Feb 29, 2024
@QilongTang QilongTang mentioned this pull request Feb 29, 2024
9 tasks
QilongTang added a commit that referenced this pull request Mar 1, 2024
* Re-work watch node display logic (#14841)

* improvements (#14497)

* [DYN-6455] Add warning text to selection node's initial warning (#14833)

* Fix PostDiff job

* Update Selection.cs

* made the warning persistent and then add removal code

* [DYN-6585] Fix Export workspace as Image option from Menu Item (#14809)

* Fix PostDiff job

* Update DynamoView.xaml

* Revert UI Blocking Function calls (#14766)

* [DYN-6354] Fix incorrect confidence score display in ML suggestions (#14829)

* Fix PostDiff job

* Update NodeAutoCompleteSearchViewModel.cs

* Update

* DYN-5745 re enable copy/paste capabilities for library searchBar (#14492)

* feat(library): keydown events binding and copy/paste for clipboard management

* refactor(library): OnPasteFromClipboard function for clipboard management including comments

* DYN-6364 Skip Lucene indexing process in UIless mode (#14523)

* skip Lucene indexing process in UIless mode

* Update comments

---------

Co-authored-by: Ashish Aggarwal <[email protected]>
Co-authored-by: Enzo Batista <[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