Esri / distance-direction-addin-dotnet

Add-in provides the ability to easily and quickly create geodesy lines, circles, ellipses and range rings.
Apache License 2.0
17 stars 22 forks source link

Label DD Circles in Pro #540

Closed lfunkhouser closed 6 years ago

lfunkhouser commented 6 years ago

Tasks to be done

Verify we are feature complete

Background information

lfunkhouser commented 6 years ago

Currently are graphics

csmoore commented 6 years ago

Current status:

  1. The layer package provided in #541 seems to work as expected
  2. Dev work still in progress in branch: https://github.com/Esri/distance-direction-addin-dotnet/tree/csm/%23541-pro-features a. The basics of this are working (screenshots below)
  3. Dev work remaining: a. more testing and adding error checking b. handle deleting the features on clear graphics c. make changes to SaveAs and any other code that may currently be using the GraphicsList

image

image

csmoore commented 6 years ago

Moving dev in progress to work on wiring up the Delete All button

BobBooth commented 6 years ago

Reviewing the checklist test (https://github.com/ArcGIS/solutions-defense-test-catalog/blob/master/Tests/Distance-Direction_ArcGIS-Pro.md)

Updates to checklist test in this PR: https://github.com/ArcGIS/solutions-defense-test-catalog/pull/144

ACueva commented 6 years ago

@BobBooth Verified your PR, waiting to remove Verify Checklist label until it can be run against the changes made to the add-in.

BobBooth commented 6 years ago

Began reviewing Solution Pages doc. Out now.

csmoore commented 6 years ago

Created PR #564 with the code changes to support circles for everything except "Save As" (now issue #559)

Note on the task above - "Share a list of questions with the team - Chris" - I couldn't remember any of the questions I had from a month ago when last working on this.

topowright-zz commented 6 years ago

First: Update website for Distance and Direction Second: Adding statement for label support to the CMS

topowright-zz commented 6 years ago

assigned @dfoll to the pull request for: https://github.com/ArcGIS/arcgis-solutions-website/pull/6479 also marked him on this task being that he is the only person on our team that can give this review.

topowright-zz commented 6 years ago

I am currently waiting on the build from @ACueva and for @dfoll to finish the updates for Military Tools CMS. Both of those tasks need to be finished before i can review the workflows for Circles.

dfoll commented 6 years ago

@topowright I updated the MT resources tab you can review the changes here http://solutionsdev.arcgis.com/defense/help/military-tools/resources/military-tools-june-2018.htm and here http://solutionsdev.arcgis.com/defense/help/distance-direction/resources/june-2018.htm for mt and dd respectively

csmoore commented 6 years ago

At scrum today it was mentioned that these should be structured differently in the geodatabase - I'm not sure where that work is captured but here is the current structure/naming (with PR #570 ):

@topowright @dfoll @ACueva @BobBooth

image

lfunkhouser commented 6 years ago

The functionality to label distance calculator will be handled in issue #569 . Removing the dependency and the task to fix this issue from this issue, Label DD Circles in Pro.