winterstein / sogive-app

The SoGive user app - See your impact
Other
2 stars 5 forks source link

Against Malaria Foundation has an issue with the cost per output #139

Closed SanjayRedScarf closed 7 years ago

SanjayRedScarf commented 7 years ago

I checked the AMF editing page, and it said that the calculated cost was £4.11 But I've looked back since, and it was using a cost of £2.09, which I believe is actually the calculated cost, contrary to what was stated in the editor. The cost of £4.11 is the override cost (not the calculated cost) and I have now entered the override cost of £4.11 into the editor and clicked on publish, and it now uses the correct cost.

I'm not sure how the stated calculated cost ended up being different from the actual calculated cost, and whether this could be an issue elsewhere.

http://app.sogive.org/#charity?charityId=against-malaria-foundation

SanjayRedScarf commented 7 years ago

Note also that the override cost field can't be expanded, so we have no way of seeing what's in that cell, which can make it harder to know what's going on.

image

winterstein commented 7 years ago

Ah yes, I see the problem. Fixed in latest code (not yet published).

SanjayRedScarf commented 7 years ago

Could there be other charities that have issues?

winterstein commented 7 years ago

The bug was a surface one -- that the calculated bit always showed the override (if set). The actual data and calculations were fine.

SanjayRedScarf commented 7 years ago

Does that mean that the override was set? If so, why didn't it feed through to be used in the app?


From: Daniel Winterstein notifications@github.com Sent: 04 September 2017 16:55:31 To: winterstein/sogive-app Cc: SanjayLikesCharities; Author Subject: Re: [winterstein/sogive-app] Against Malaria Foundation has an issue with the cost per output (#139)

The bug was a surface one -- that the calculated bit always showed the override (if set). The actual data and calculations were fine.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/winterstein/sogive-app/issues/139#issuecomment-326992932, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARuzWUztArHRLdvhYHT5gH5WLSUMuiNPks5sfB1zgaJpZM4PL9ss.