[NightTigress]

Bug in milestone calculation in Walking Club

The "minutes walked", the milestone image, the "next milestone" text, and the "to go" text all tell one story.
The progress bar, the fact that it lists "25,000" under "milestones passed", and the celebration thread don't seem to agree.

Update: Now we have the "25,000" milestone passed twice in there and a second celebration thread.

 

[neilarey]

@NightTigress Good catch and thank you for the screenshot, it made this a two-minute hunt. The club total can go down when someone removes a walk, and the milestone code did not know that. It saw the total slip under 25,000, decided 25,000 was the next target again and celebrated it a second time when the club crossed it. Fixed now: once the club passes a milestone it stays passed, the bar and the "to go" number agree with each other again and the second thread is gone. The two roll calls were merged so nobody who carried the club there was dropped.