tperrier / mwachx

Mobile SMS management application for maternal health
Apache License 2.0
2 stars 8 forks source link

Count of pending visits is broken #274

Closed bderenzi closed 9 years ago

bderenzi commented 9 years ago

Marked a visit as happened, but the count on the homepage didn't change. In general as we moved forward and backward in time it seemed broken. E.g., when visits were > 7 days overdue, they're not in the list but still impact count

tperrier commented 9 years ago

I can't replicated this? Everything seems to work well on my computer. This might be an issue with timezones? There is a issue with the visits where if you go back in time a visit marked as seen will not reappear - I figure since time will not go backwards this is ok.

However, if the Home visit count and Visits page displayed visits are not matching up that is bug.

On Fri, Oct 16, 2015 at 6:31 AM, bderenzi notifications@github.com wrote:

Marked a visit as happened, but the count on the homepage didn't change. In general as we moved forward and backward in time it seemed broken. E.g., when visits were > 7 days overdue, they're not in the list but still impact count

— Reply to this email directly or view it on GitHub https://github.com/tperrier/mwachx/issues/274.

bderenzi commented 9 years ago

Just pulled the latest changes, refreshed the DB and was able to replicate the issue. Steps I took:

  1. pulled latest data
  2. python manage.py reset_db -P 15
  3. gulp
  4. ./manage.py runserver

Then once the app was up:

  1. Login; 0 new visits
  2. Press the forward date once: 1 new visit
  3. Press the forward date a second time: 2 new visits listed mainmenu
  4. Go to pending vistis page; only one visit is listed. visits