-
Notifications
You must be signed in to change notification settings - Fork 901
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Database connection toolbar missing for sql scripts #16436
Comments
How are you opening the file? (command line, file -> open, other?) If you aren't opening it via file -> open, does the issue still repro when closing it and re-opening with file -> open? |
⌘+O keyboard shortcut
It does yes. Interestingly, files which were open before the update are still showing the menu. |
We need more info to debug your particular issue. If you could attach your logs to the issue (ensure no private data is in them), it would help us fix the issue much faster. There are two types of logs to collect: Console Logs
Application Logs
|
Hm, well I was able to repro this behavior a couple times myself but now it seems to be working as expected.... Could you try :
Does the editor still show without the toolbar? |
For anyone else hitting this issue, could you please provide the following information :
|
@jesman3 Could you try :
|
@Charles-Gagnon same for me, it's the same even if I've restarted the Mac. Where are the cache files stored on a Mac and I'll give that a go |
@Charles-Gagnon |
@Charles-Gagnon it seems happier now but I'll let you know if it continues to be an issue tomorrow |
@Charles-Gagnon I believe it is a User installation but I am not sure. I can duplicate this by opening any .sql file as it will not have the connection toolbar and I cannot seem to figure out how to get the .sql file to connect to a database. |
@Charles-Gagnon One other note is that I download a pre-release version of Azure Data Studio about a month ago to test for a bug fix regarding an array slice error and I noticed this same SQL toolbar behavior in the pre-release version but I didn't think too much of it as I was just testing the array slice error. I still have that pre-release version installed but I don't use it. Also, my co-workers who still have version 1.30 are not experiencing this issue. |
Thanks for the confirmation @timgaunt - my understanding of the issue is that as long as your cache isn't cleared this issue shouldn't happen for you anymore. But definitely let me know if that's not the case. @jesman3 The Help -> About window will have that information Can you get me your logs per these instructions? It's unclear why your instance is behaving differently than mine during my testing and that might have something useful in it Log Collection Instructions There are two types of logs to collect: Console Logs Open Developer Tools (Help -> Toggle Developer Tools) Open command palette (Click View -> Command Palette) |
@jesman3 If you get a chance could you try out this private build to see if it resolves your issue? https://sqltools.blob.core.windows.net/public/azuredatastudio-win32-x64.zip Just unzip that and run the |
I have the exact same symptoms as @jesman3, I use the user setup version. The insider build you have provided does indeed fix the issue for me (the offending .sql files have to be reopened though) |
@Charles-Gagnon I did not have the same luck as @dot-fs the insiders version did not resolve the issue. I have attached a gif showing the easiest flow to duplicate the issue. The only thing I notice is that the language on the status bar at the bottom changes from MSSQL when the toolbar is visible to SQL when the toolbar is not visible. |
Interestingly, just using the insider build seems to have fixed the installed version for me. |
FYI, I finally just did a full uninstall and re-install and everything is working fine in version 1.31. Sorry you have one less test subject now. Thanks for your help!!! |
Thanks for letting me know @jesman3! There definitely seems to be some sort of caching issue, if anyone else tries uninstalling/reinstalling and is still seeing this issue then please let me know. We'll have a hotfix out for fixing some of the other issues that people are seeing with this, but the suggestions above in this thread should mostly resolve any problems people are running in to currently with my current understanding of the specific issues being hit here. |
This has also fixed my issue. The other problem that I experienced at the same time (and still with this fix) is any of my old saved templates stopped being recognised as queries and were opening as text pages. (they didn't have .sql as their file type but worked fine) When you create 'new file' from within explorer it's creating a 'Plain Text' page, where as before this would open anew query. after creating this, if you then change the language type to SQL from plain text, it then locks the page and you can't type anything. If you then rename the page to include the .sql file extension at the end, close it and re open it, it displays as it should |
@TunnyF24 That's expected behavior - we recognize things as queries based on their file extension and so if they don't have a .sql extension then they'll just open as plain text. So when creating a new file make sure to add the .sql extension to the end (or add it later and close/reopen as you found out) Changing the language mode doesn't change the editor - so a text editor set to the SQL language will not be displayed as a query editor. It's something we'd like to fix at some point - but I believe this has always been the behavior for files originally opened as text files. |
Having this same issue on Mac. Tried uninstall / reinstall. SQL files open as plain text files (no connection toolbar available). I see the private build available for Windows, but any love for us Mac users? Quick fix is open .SQL file (has no connection toolbar), open new tab (has toolbar by default), copy/paste script from .SQL tab to new tab. Only started having this problem a couple of days ago - an issue with the new build/release on 7/21??? From About ADS: |
@emilhizer The latest insiders build has the fix for this. Let us know if that still doesn't solve your issue. (Note that you will still likely need to close and re-open any documents that were persisted after the update) We'll be releasing a hotfix for this in the next day or two for the stable version. |
Thanks @Charles-Gagnon - works perfectly. And thanks for the direct link to the insiders build. |
The stable hotfix for this issue has gone out - please update to 1.31.1 and if you're still seeing issues let me know! |
Did you close and re-open the tabs? If that still isn't working try this :
|
I tried it but it doesn't work. |
Tried what? Both options? Some people also said that they had to uninstall and reinstall to get it to pick up the fix properly - so would be worth trying that if you're able. Otherwise could you list the exact steps you're taking to open this file? |
Could you
Log Collection Instructions There are two types of logs to collect: Console Logs Open Developer Tools (Help -> Toggle Developer Tools) Open command palette (Click View -> Command Palette) Upload both sets of logs to this issue. |
@Charles-Gagnon Thank you, please see the files below: |
Thanks! Unfortunately that still wasn't enough to figure out what was going on here - I uploaded a new version with even more logging. So if you could download that and follow the same steps I'd appreciate that! Link is the same : https://sqltools.blob.core.windows.net/public/azuredatastudio-win32-x64.zip (also I removed the logs from your message since I no longer need them) |
@Charles-Gagnon Hi I have upgraded my ADS to version 1.33.1, but the issue is still there. Would you please help? |
@MangoZMR Does this continue to happen even if you close the query and open a new one? |
@Charles-Gagnon There is no toolbar even I close the query and open a new one. Thank you. |
Alright, here's another private build with additional logging if you don't mind trying it out
Log Collection InstructionsThere are two types of logs to collect: Console LogsOpen Developer Tools (Help -> Toggle Developer Tools) Application LogsOpen command palette (Click View -> Command Palette) E-mail me the logs (chgagnon@microsoft.com) or upload them to this issue |
I'll leave this here, maybe it will help somebody. |
Steps to Reproduce:
Actual
Expected
Does this issue occur when all extensions are disabled?: Yes
The text was updated successfully, but these errors were encountered: