Skip to content

Create "Add database" button in DB panel #1882

Closed
norascheuch wants to merge 1 commit intomainfrom
shati-nora/add-remote-repos
Closed

Create "Add database" button in DB panel #1882
norascheuch wants to merge 1 commit intomainfrom
shati-nora/add-remote-repos

Conversation

@norascheuch
Copy link
Copy Markdown
Contributor

To add new databases to the new DB panel we want to have an "Add database" button (using the "add" icon), which lets a user add a new database or an owner item through the UI.

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 16, 2022 08:13
@norascheuch norascheuch requested review from a team as code owners December 16, 2022 08:13
Comment thread extensions/ql-vscode/src/databaseFetcher.ts
Copy link
Copy Markdown
Contributor

@shati-patel shati-patel left a comment

Choose a reason for hiding this comment

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

Thanks! Looks good overall ❤️ Just left some comments that we mostly already talked about 😸

Comment thread extensions/ql-vscode/src/databaseFetcher.ts Outdated
Comment thread extensions/ql-vscode/src/databases/git-hub-url-identifier-helper.ts Outdated
Comment thread extensions/ql-vscode/src/databases/git-hub-url-identifier-helper.ts Outdated
@norascheuch
Copy link
Copy Markdown
Contributor Author

PR for the refactor: #1888

@norascheuch norascheuch force-pushed the shati-nora/add-remote-repos branch from 5bb3670 to 7241e31 Compare December 20, 2022 10:37
@shati-patel shati-patel deleted the shati-nora/add-remote-repos branch December 20, 2022 16:32
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