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

Seeking community feedback for Import/Export Data-Tier Application Framework Wizard extension #3171

Open
yualan opened this Issue Nov 8, 2018 · 18 comments

Comments

Projects
None yet
7 participants
@yualan
Copy link
Member

yualan commented Nov 8, 2018

As part of bringing over key features from SSMS, the team is invested in bringing over DacFx support in Azure Data Studio, starting with the Export and Import bacpac wizards in SSMS. According to telemetry, these are the second most used wizards in SSMS after the Import Flat File Wizard.

By bringing over these experiences, we have an opportunity to improve the existing experience so that users have a better idea of knowing when to use Extract/Export/Import or more guidance for when to use DACPACs or BACPACs.

Here are some GIFs of the current experience.

Import
importbacpac

Export
exportbacpac

We would appreciate your help by letting us know your thoughts of the current experience, and suggestions you have in mind as we continue to build out this feature.

You can keep track of current work here: #3163

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Nov 8, 2018

@yualan Thanks for this. Can you share how to test this. I tried cloning and building branch feature/dacfxexport but the export bacpac did not succeed. Upon clicking Export button there was no task progress displayed and no bacpac file was created.

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Nov 8, 2018

Bug: I found that when I change the bacpac location, the default location still shows on the last page of the wizard.

Suggestion: it would be nice if Import\Export location defaulted to the current root path open in the file explorer.

Thanks for listening.

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Nov 8, 2018

Are there plans to include the Schema Compare like DacFx Script action?
How about including a way to execute scripts in sql command mode?

Thanks for listening.

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Nov 10, 2018

@yualan , I'm Looking forward to providing feedback to improve the cmd line experience for Extract, Compare, Update

This is what we're working with now.

alt demo

Would love to see this become a wizard experience in ADS.

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Nov 10, 2018

It appears the DacFx Script action is not included here: Microsoft/sqltoolsservice#728

Question: Any chance of adding the script action before this merges?

Scripting changes rather than publishing directly to database is really a important scenario...essential for maintaining the DEV to DBA relationship. :-)

Thanks for listening.

@yualan

This comment has been minimized.

Copy link
Member

yualan commented Nov 13, 2018

Hi Josh, love to hear your enthusiasm. Would you be interested in joining a call with our developers and try the bits yourself? You can ask your questions on there, and then we can share the outcomes on this thread. Feel free to contact me at yu.alan@microsoft.com

@armordba

This comment has been minimized.

Copy link

armordba commented Nov 13, 2018

This would be wonderful since we use SSDT heavily in our shop. We run CI/CD. Just install Azure Data Studio, looking to explore and learn more. Also loading a Linux container on my Mac.

@yualan

This comment has been minimized.

Copy link
Member

yualan commented Nov 13, 2018

Thanks Tom. Do you use this wizards in SSMS currently? Any feedback on the experience shown in the GIF?

@kburtram kburtram modified the milestones: December Release, Backlog Nov 13, 2018

@armordba

This comment has been minimized.

Copy link

armordba commented Nov 15, 2018

@yualan

This comment has been minimized.

Copy link
Member

yualan commented Nov 15, 2018

Hi @armordba , what are the main features you are looking for from SSDT?

@armordba

This comment has been minimized.

Copy link

armordba commented Nov 15, 2018

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Dec 5, 2018

@yualan & @kisantia

It appears the DacFx Script action is not included here: Microsoft/sqltoolsservice#728

Question: Any chance of adding the script action before this merges?

Scripting changes rather than publishing directly to database is really a important scenario...essential for maintaining the DEV to DBA relationship. :-)

Thanks for listening.

cc: #3163

@kisantia

This comment has been minimized.

Copy link
Contributor

kisantia commented Dec 5, 2018

@joshbooker I will look into adding the Generate Script action to the deploy scenario, but it won't make it into the December release.

Thanks,
Kim

@joshbooker

This comment has been minimized.

Copy link

joshbooker commented Dec 6, 2018

@kisantia thanks so much for your reply. Looking forward to it. Have a great day!

@Alain1405

This comment has been minimized.

Copy link

Alain1405 commented Jan 4, 2019

EDIT: Ignore this comment

Version 1.2.4
Commit 1ebfd0235690ad044ed2874a02e65188f5fea01c
Date 2018-11-02T21:36:55.143Z
VS Code 1.26.1
Shell 2.0.9
Renderer 61.0.3163.100
Node 8.9.3
V8: 6.1.534.41
Architecture x64

I'm trying to import an Azure SQL backup in bacpac format into my local SQL Server on my Mac.
I've installed the import extension yet the Data-tier application wizard is not available, only the Flat file import.
Am I missing something? The command is not available in the commands palette neither

screen shot 2019-01-04 at 3 48 21 pm

@Alain1405

This comment has been minimized.

Copy link

Alain1405 commented Jan 4, 2019

Actually, I was supposed to use an insider build. Using1.3.6-insider build, however, I encounter another issue, the Next button stays obfuscated once I try to import a bacpac file.

screen shot 2019-01-04 at 5 36 43 pm

@Alain1405

This comment has been minimized.

Copy link

Alain1405 commented Jan 5, 2019

I've downloaded the source for version 1.4.0 and built on Mac, to find that I had the same issue of Next button being disabled. I then opened Chrome Developer Tools and manually removed the CSS disabled configurations. Once the button was enabled, the import worked out.

@kisantia

This comment has been minimized.

Copy link
Contributor

kisantia commented Jan 9, 2019

@Alain1405 thanks for finding this bug! I checked in the fix #3691 and it will be included in the next release of ADS.

A workaround in the current version is to click the Previous button to go back to the Step 1 page, then click the Next button to go back to the 'Select Import Bacpac Settings' page and the Next button should be enabled if the required fields are filled in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment