Skip to content

Create method that returns selected DbItem#1830

Merged
norascheuch merged 7 commits intomainfrom
nora/selected-db-item
Dec 6, 2022
Merged

Create method that returns selected DbItem#1830
norascheuch merged 7 commits intomainfrom
nora/selected-db-item

Conversation

@norascheuch
Copy link
Copy Markdown
Contributor

When running a query we need to know which DbItem is selected in the new DB panel. This PR adds a method to the DbManager that returns the currently selected item.

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@norascheuch norascheuch marked this pull request as ready for review December 2, 2022 16:54
@norascheuch norascheuch requested a review from a team as a code owner December 2, 2022 16:54
Copy link
Copy Markdown
Contributor

@charisk charisk left a comment

Choose a reason for hiding this comment

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

This is great, thank you! Love seeing all the tests too.

Comment thread extensions/ql-vscode/src/databases/db-item-selection.ts Outdated
Comment thread extensions/ql-vscode/src/databases/db-item-selection.ts
Comment thread extensions/ql-vscode/test/pure-tests/databases/db-item-selection.test.ts Outdated
Comment thread extensions/ql-vscode/src/databases/db-manager.ts Outdated
@norascheuch norascheuch force-pushed the nora/selected-db-item branch from c08ad4b to 7a2f976 Compare December 5, 2022 15:22
Copy link
Copy Markdown
Contributor

@charisk charisk left a comment

Choose a reason for hiding this comment

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

Thanks for making the changes and thinking about readability.

Comment thread extensions/ql-vscode/test/pure-tests/databases/db-item-selection.test.ts Outdated
norascheuch and others added 2 commits December 5, 2022 16:57
…on.test.ts

Co-authored-by: Charis Kyriakou <charisk@users.noreply.github.com>
@norascheuch norascheuch enabled auto-merge December 6, 2022 08:29
@norascheuch norascheuch merged commit c436688 into main Dec 6, 2022
@norascheuch norascheuch deleted the nora/selected-db-item branch December 6, 2022 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants