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

Coordinate Output Settings/Configuration not saved at Pro 2.4 #605

Closed csmoore closed 4 years ago

csmoore commented 5 years ago

On Pro 2.4 when you add and configure Output Coordinate formats, these should be saved in-between Pro runs until you hit the Reset button. At Pro 2.4 these are no longer saved in-between runs.

Similar to Issue: https://github.com/Esri/military-symbology/issues/374 - Pro 2.4 no longer allows files to be saved to the Assembly Cache (C:\Users\{user}\AppData\Local\ESRI\ArcGISPro\AssemblyCache):

Repro steps:

  1. On Pro 2.4
  2. Launch Pro, Create a project, Launch Coordinate Conversion Addin
  3. In the Output Group, use the "+" Add button to "Add New Output Coordinate Format"
  4. Add several new formats (ex: DD, MGRS)
  5. Close Pro
  6. Launch Pro, reopen project oordinate Conversion Addin
  7. There are no Output Formats in the Output Group (these should be the same as those added in Step 3),

image

csmoore commented 5 years ago

Addressed in PR #606

dfoll commented 5 years ago

verified issue exists using Pro 2.4 B1 with MT (released build 227)

dfoll commented 5 years ago

verified issue is fixed using Pro 2.4 B1 with CC (build 95)

dfoll commented 5 years ago

because we have never released on 2.4 and this issue is 2.4 specific, I do not think that we need to doc this issue

lfunkhouser commented 5 years ago

now saved to ...\Documents\ArcGIS

csmoore commented 4 years ago

A later PR ( #612 ) seems to have accidentally overwrote this fix ( in #606 ) - at https://github.com/Esri/coordinate-conversion-addin-dotnet/pull/612/files#diff-72fc8a70d9ede6e7ce33f30c4e19a45eL158

Just need to reapply this fix

dfoll commented 4 years ago

Observed problem with MT 344 on Pro, did not observe on ArcMap.

With CC build 173 (csm-605-redo) this looks good on Pro 2.4. and ArcMap 10.4.1 (functionality worked before and after on ArcMap)

dfoll commented 4 years ago

Rebuilt CC and MT from dev after #635 was merged