Skip to content

Remove recent from diaper driver calculation#1031

Merged
seanmarcia merged 1 commit intorubyforgood:masterfrom
JuarezLustosa:fix/month_dash_distribution
Jun 1, 2019
Merged

Remove recent from diaper driver calculation#1031
seanmarcia merged 1 commit intorubyforgood:masterfrom
JuarezLustosa:fix/month_dash_distribution

Conversation

@JuarezLustosa
Copy link
Copy Markdown
Contributor

Resolves #1029

Description

The Diaper drive calculation was coming from recent_donations, so was getting only 3.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Acceptance Test plus manual

@seanmarcia seanmarcia merged commit 384c9c2 into rubyforgood:master Jun 1, 2019
@seanmarcia
Copy link
Copy Markdown
Member

Thanks for this!

@armahillo
Copy link
Copy Markdown
Collaborator

@JuarezLustosa Hey! Thanks for writing this :)

I've actually got a PR (#1030) that resolves this issue as well in addition to fixing our Dashboard system specs, but I would like to see if we can still get you some commit credit for taking this on.

Could you open a new PR that puts your branch JuarezLustosa:fix/month_dash_distribution against rubyforgood:972-incorrect-diaper-drive-numbers? We'll have a few merge conflicts there, but I can resolve those, and this way you still get contribution credit.

Thank you!

@JuarezLustosa JuarezLustosa deleted the fix/month_dash_distribution branch June 3, 2019 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dashboard Not Adding Statistics Correctly *Please Prioritize This*

3 participants