Skip to content

[Cherrypick] Integrating ML node autocomplete. (#13496)#13533

Merged
reddyashish merged 1 commit intoDynamoDS:RC2.17.0_masterfrom
reddyashish:2.17-cherrypicking
Nov 17, 2022
Merged

[Cherrypick] Integrating ML node autocomplete. (#13496)#13533
reddyashish merged 1 commit intoDynamoDS:RC2.17.0_masterfrom
reddyashish:2.17-cherrypicking

Conversation

@reddyashish
Copy link
Collaborator

Purpose

Cherrypicking #13496

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

[Cherrypick] Integrating ML node autocomplete. (#13496)

Reviewers

@QilongTang

* Integrate ML Node-Autocompletion,

* Update NodeModel.cs

* Update NodeModel.cs

* some changes

* more changes

* Update NodeAutoCompleteSearchControl.xaml

* Fix edge cases and UI improvements.

* update

* Update NodeAutoCompleteSearchControl.xaml

* fix highlighting items.

* fix

* Port name for variable input nodes.

* Add base test.

* addressing comments.

* Fix the edge cases for search results.

* Clean up code

* Update NodeAutoCompleteSearchViewModel.cs

* refactoring.

* updates

* Use IDSDK token for authentication

* updates

(cherry picked from commit 34fe0f1)
@reddyashish reddyashish merged commit d9302ab into DynamoDS:RC2.17.0_master Nov 17, 2022
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.

1 participant