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.

Sign up for GitHub

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

Jump to bottom

Checks for lat/lon before making api calls on project view #608

Open

ostermanj wants to merge 1 commit into focusconsulting:dev from ostermanj:dev

Collaborator

ostermanj commented Oct 18, 2017

Avoids making bad API calls with null lat or lon data of project views. RE issue #604 .

Outstanding issue to resolve:

We could try to resolve a lat/lon based on project address on the front end, but
Isn't there a reason we haven't done that on the back?
If we do neither 1 or 2 for now, we should display to the user that the data is missing rather than just having the fields not populate.

Thought?


          Checks for lat/lon before making api calls

6044fd8

ostermanj requested a review from NealHumphrey

October 18, 2017 18:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet