Flat File Import preview "Refresh" button is unnecessary #6611
Labels
Milestone
Comments
|
I remember we had a reason for getting this. I think the idea was if the user didn't like how ADS treated the data they can edit the file and refresh. But I agree with you it seems like an extreme edge case! |
|
IIRC, it also was with the expectation that we would eventually provide options to configure the PROSE library on the preview page. I'll move this to the backlog for now. |
aasimkhan30
added a commit
that referenced
this issue
May 1, 2020
- If a PG database is active sending an error - Removed refresh button #6611
aasimkhan30
added a commit
that referenced
this issue
May 6, 2020
…#10248) * - Removed PG connections in the new connection wizard - If a PG database is active sending an error - Removed refresh button #6611 * Added back the refresh * -Using getConnection only once. * -Fixed the error message -Fixed the bugs mentioned in the PR Co-authored-by: Aasim Khan <aaskhan@microsoft.com>
|
This should be fixed on current builds. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
masterSteps to Reproduce:
Expected: Refresh button does something meaningful. Perhaps refreshing the data after modifying it or with a different schema or something.
Actual: The refresh button re-reads the file without modification. Not sure the use case. I guess if the user manually changed the file on disk while the wizard was running, which seems like a corner-case scenario.
Note: for reference SSMS does not provide a Refresh button on this screen.
ADS screen
SSMS screen
The text was updated successfully, but these errors were encountered: