* Connection List UI Improvements
- Add symbols for Profile vs MRU (MRU will be added in future update)
- Connection is now 1-line
- Add server name to the description. Will update to skip for Profiles in a later update
* Connection should display actual DB name
- On connecting to a server with no DB specified, we will actually get a connection to Master or some default DB.
- This database should be what's displayed to the user instead of <connection default>, and used in the file mapping
- Updated status view to also use DOMAIN\USERNAME for Integrated auth accounts instead of <prompt>
- Added basic unit tests to cover this scenario
* Handle Profile with no name
- Profile names are now optional. All calls (get, save, remove) support this with tests
- Handle difference between MRU and Profile in pick list
- Formatting of Connection list in UI matches functional spec
* Fixes based on PR comments
- Moving isSameProfile function to utils. It's hard to put this directly on an interface as it makes simple object serialization/deserialization to the interface more complex
- converting function in then callback to a lambda function
* Restore type definition to lambda
* code changes for save results as csv
* config changes and bug fixes
* Retrieve the right resultSet after batch execution
* Retrieve the right resultSet after batch execution
* code clean up
* Delete .BROWSE.VC.DB-shm
* Delete .BROWSE.VC.DB-wal
* fix merge error
* fix merge error
* fixed merge error - removed viewChild decorator and moved html
* made filepath compulsory
* reset defaults
* remove strict
* removed JSON message and set http status
* remove resize call in tab.ts because of bug with small result sets
* Remove Save command
* Fix tests for Mac/Linux
* Remove dead code
* Change class name to ResultsSerializer
* Remove new lines
* Revert "remove resize call in tab.ts because of bug with small result sets"
This reverts commit 6b63a4db28.
* Connection should display actual DB name
- On connecting to a server with no DB specified, we will actually get a connection to Master or some default DB.
- This database should be what's displayed to the user instead of <connection default>, and used in the file mapping
- Updated status view to also use DOMAIN\USERNAME for Integrated auth accounts instead of <prompt>
- Added basic unit tests to cover this scenario
* Added doc comments to the file
* CredentialStore support via ServiceLayer
- Removed Node-based credential implementation as it send password over the command line
- Replaced with implementation via the ServiceLayer
- Improved error handling to catch rejected Thenables
- Removed no longer used npm dependencies
* Save Password fixes
- Renamed RecentConnections to ConnectionStore to accurately reflect status (it stores Profile info and in the future will also have MRU)
- Added a unit test to cover not saving password in the settings. Will update to include in the secret store instead in a future checkin
* Save and Remove password from credential store
- Password is now saved to credential store and removed from it when RemoveProfile is called
* ConnectionUI constructor requires ConnectionStore
* Fix casing for Credential objects
* Multiple fixes for Connection Behavior
- Integrated authentication handled correctly in the Create Profile workflow
- Error Handling in the main events so that rejected promises are not lost
- Handle Escape from Connection workflows correctly
* Use SqlLogin as authenticationType
* Use VSCodeWrapper for calls to ShowErrorMessage
* Update Nuget Dependency to 0.0.8
Leila Lali
Anthony Dresser
Kevin Cunnane
Sharon Ravindran
Anthony Dresser
Mitchell Sternke
Mitchell Sternke
Karl Burtram