Esri / coordinate-conversion-addin-dotnet

Addin for ArcMap and Pro for convenient coordinate conversion in Desktop.
Apache License 2.0
45 stars 20 forks source link

Import functionality should look through an Excel workbook to allow the user to select which worksheet to import #588

Closed dfoll closed 4 years ago

dfoll commented 5 years ago

Current behavior

Using the add-in to import an Excel file only allows the user to select a workbook. By default the first worksheet in the workbook is the one used for import.

Expected behavior

The add-in should allow the user to select which worksheet they want to import from a workbook. ArcGIS natively allows the user to search through a workbook when adding data, however the add-in only allows the user to select a workbook.

dfoll commented 5 years ago

Excel to table does not allow user to easily select a sheet image image

If we want to populate the list of sheets we need to revert our decision to use the gp tool and use npoi or interop. if we use the interop option, there will be a requirement for excel to be installed.

dfoll commented 5 years ago

For this release we will match the functionality of the gp tool that is used. We will allow the user to explicitly specify a worksheet. The ability to select a worksheet by browsing is something we will evaluate in the future.

dfoll commented 5 years ago

We are not currently going to add the functionality to browse to/use as specific worksheet. We will add a note that when using a .xlsx that the first worksheet in the workbook is the one that is imported. Possibly add to the note that the user can use the core tool if they must use a worksheet that is not the first one.

Issue will be closed once note is added

reason for strikeouts is here

dfoll commented 4 years ago
dfoll commented 4 years ago

Leaving this open temporarily while we get confirmation about the level of effort for allowing a user to browse to a specific worksheet.

I had discussions with @saip about this in the past, but the reason we cannot do this is currently escaping me, leaving open until we get clarification.

@csmoore for awareness or any possible input

dfoll commented 4 years ago

Doc QC work:

lfunkhouser commented 4 years ago

Verified note added to Import coordinates topic: When importing a .xls/.xlsx file the first sheet in the workbook is used as input.

Verified on Dev site for CC and MT and PDFs CC 175 and MT 345

Reopening... https://github.com/Esri/coordinate-conversion-addin-dotnet/issues/588#issuecomment-534754376

dfoll commented 4 years ago

"As designed" the functionality is not designed to be able to use a sheet from a workbook other than the first sheet. If a user brings to us a need to use a sheet that is not the first sheet in a workbook and the work around of editing the workbook to make it the first sheet is not an option, we will open a new issue for that functionality at that time