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

Location Support #23

Merged
merged 7 commits into from
Feb 7, 2024
Merged

Location Support #23

merged 7 commits into from
Feb 7, 2024

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Jan 25, 2024

  • Add support for inputting a Lat/Lon when inputting data and storing this data in the database
  • Added support for adding the gridCellId as well
  • Updated the GET /recording endpoint to support returning back geoJSON for the gemotry location of the location
  • Added a quick sample script to go from Lat/Lon to gridCellId automatically, this will be removed in final version

TODO:

  • Map support for the lat/lon locations to make it easier to select the location. GeoJS showing the location of the current latitude longitude
  • Similar to the input map, for each recording we may want information for the location of the recording based on geoJSON values.
  • Make it so there is a two-way binding between lat/lon and editing it in the input field will update the map location

references:

Demo:

recordingLocationSmall.mp4

@BryonLewis BryonLewis marked this pull request as ready for review February 6, 2024 17:29
@BryonLewis
Copy link
Collaborator Author

I've included in this PR a script to attempt to transfer from lat/lon to a gridCellId. I haven't integrated yet but it's there for future reference if we decide to have that functionality.

@BryonLewis BryonLewis merged commit ce43eb8 into main Feb 7, 2024
6 checks passed
@BryonLewis BryonLewis deleted the recording-location branch February 22, 2024 15:50
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.

1 participant