Skip to content

fix the address check on foodsaver's dashboard.

Peter Tönnies requested to merge Feature/PT-fixAddressCheckerOnFSDashboard into master

Closes none

What does this MR do?

if lat or lon of the address fields are empty or 0 or not set, the code was meant to through a warning and guide to the settings. Like this, all foodsavers are forced to set an address. As we try to nudge correct addresses from foodsavers at the moment and have some more explanations and a new geo-locator in place, this is a good moment to introduce this fix here.

How confident are you it won't break things if deployed?

Very. But it will send many foodsavers to the settings for the next visit.

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Peter Tönnies

Merge request reports