Skip to content

Conversation

@reddyashish
Copy link
Contributor

Purpose

This PR is to cherry-pick #10230 into RC2.5.0_master branch.

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.

Reviewers

@mjkkirschner

…d API (DynamoDS#10230)

* DYN-2349

* Adding events and fixing the crash that was happening due to selection/deselection of the menu item.

* Change to internal

* Moving the Close tab event from ViewLoadedParams to WorkspaceDependencyView call

* Adding extension tab name as an input parameter to the Close Extension event

* Adding null check for WorkspaceReferencesMenuItem

* Making the extension name string a constant

* Removing the extra line
@reddyashish reddyashish changed the title [DYN-2349] Close workspace references extension tab by user action an… [Cherry-pick][DYN-2349] Close workspace references extension tab by user action an… Dec 19, 2019
@mjkkirschner mjkkirschner added the LGTM Looks good to me label Dec 19, 2019
@mjkkirschner
Copy link
Member

LGTM

@reddyashish reddyashish merged commit 0b9616e into DynamoDS:RC2.5.0_master Dec 19, 2019
@reddyashish reddyashish deleted the 2.5-cherrypicking branch November 23, 2022 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM Looks good to me

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants