Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Round nactive in the header #1400

Merged
merged 1 commit into from
Sep 6, 2013
Merged

Round nactive in the header #1400

merged 1 commit into from
Sep 6, 2013

Conversation

chadwhitacre
Copy link
Contributor

In #1377 we rounded the dollar amount. Let's round the number of users
as well.

@chadwhitacre
Round nactive in the header
58cb4bf 
In #1377 we rounded the dollar amount. Let's round the number of users
as well.
@chadwhitacre chadwhitacre mentioned this pull request Sep 6, 2013
Merged
wyze added a commit that referenced this pull request Sep 6, 2013
@wyze
Merge pull request #1400 from gittip/round-users
71784dd 
Round nactive in the header
@wyze wyze merged commit 71784dd into master Sep 6, 2013
@wyze wyze deleted the round-users branch September 6, 2013 14:10
@tshepang
Copy link
Contributor

tshepang commented Sep 6, 2013

I think rounding off is a bad idea. It gives an inaccurate picture. If you insist, then we should have something like ~1,700 USERS EXCHANGING ~$5,800 PER WEEK. When I saw this on the home page, I was actually wondering what's the chances that at last count, exactly $5,800 was exchanged. Maybe you guys were thinking it would be obvious?

@chadwhitacre
Copy link
Contributor Author

@tshepang New ticket, please.

@tshepang tshepang mentioned this pull request Sep 6, 2013
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chadwhitacre @tshepang @wyze