Skip to content
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

GH 3042 Add MGRS coordinate format #3062

Merged
merged 3 commits into from
Sep 8, 2020

Conversation

ysf-simsoft
Copy link
Contributor

@ysf-simsoft ysf-simsoft commented Jun 15, 2020

Added MGRS coordinate format support. System admin can define which format to use through the dictionary. Currently two formats are supported: MGRS and LAT_LON. LAT_LON is the default when no format is specified or an unknown format is defined in the dictionary.

Coordinates are still stored as lat,lon values. However, when displaying/editing locations we will convert between lat,lon and the specified format behind the scenes.

Release notes

Closes #3042 and 1st and 2nd items from #244

User changes

  • Users will see coordinates in specified format

Super User changes

  • none

Admin changes

  • While editing/creating a location, admin will see form field(s) for coordinates according to the format defined in anet.yml.

System admin changes

  • System admin can define which coordinate format to be used throughout the UI in anet.yml.

  • anet.yml needs change

  • db needs migration

  • documentation has changed

  • graphql schema has changed

Checklist

@lgtm-com
Copy link

lgtm-com bot commented Jun 15, 2020

This pull request introduces 1 alert when merging 3f167de into 57532b7 - view on LGTM.com

new alerts:

  • 1 for Superfluous trailing arguments

@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch from 3f167de to 1396fee Compare June 15, 2020 10:38
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch from 1396fee to 23c75c9 Compare June 22, 2020 12:54
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch from c131dda to 79f0547 Compare June 29, 2020 19:36
@ysf-simsoft ysf-simsoft requested a review from maradragan June 30, 2020 09:10
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch 2 times, most recently from 05d7ab6 to ef8fce5 Compare July 9, 2020 18:26
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch 2 times, most recently from e4fdcfd to 7436628 Compare July 24, 2020 15:51
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch from 7436628 to 8dca8d3 Compare August 22, 2020 17:20
Coordinates are still stored as Lat,Lon format. No changes have been
made in backend or DB Schema. We simply convert between MGRS
and Lat,Lon while displaying/editing coordinates.
@ysf-simsoft ysf-simsoft force-pushed the GH-3042-add-mgrs-coordinate-format branch from 8dca8d3 to ed28f0a Compare September 8, 2020 06:22
@VassilIordanov VassilIordanov merged commit 715fe3a into candidate Sep 8, 2020
@VassilIordanov VassilIordanov deleted the GH-3042-add-mgrs-coordinate-format branch September 8, 2020 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add MGRS coordinate format
3 participants