WIP: Stats email v2 #1311

oogetyboogety · 2018-05-30T02:08:33Z

Description

Checklist

linter status: 100% pass
changes don't break existing behavior
commit message follows commit guidelines

Affected core subsystem(s)

Testing

Refers/Fixes

oogetyboogety · 2018-05-30T02:09:22Z

@owocki This just need the hardcoded text to be localized. Otherwise, this should be finished, unless there are CSS changes to be included?

stickler-ci · 2018-05-30T02:21:05Z

app/dashboard/models.py

@@ -1065,6 +1082,175 @@ def stats(self):
            (success_rate, 'Success Rate'),
        ]

+    @property


C901 'Profile.get_quarterly_stats' is too complex (17)

stickler-ci · 2018-05-30T02:21:05Z

app/dashboard/models.py

+            for bounty in funded_bounties:
+                hourly_rate = bounty.hourly_rate
+                if hourly_rate:
+                    total_funded_hourly_rate += bounty.hourly_rate 


W291 trailing whitespace

stickler-ci · 2018-05-30T02:21:05Z

app/dashboard/models.py

+                    hourly_rate_bounties_counted += 1
+            funded_bounty_fulfillments = []
+            for bounty in funded_bounties:
+              fulfillments = bounty.fulfillments.filter(accepted=True)


E111 indentation is not a multiple of four

stickler-ci · 2018-05-30T02:21:05Z

app/dashboard/models.py

+            funded_bounty_fulfillments = []
+            for bounty in funded_bounties:
+              fulfillments = bounty.fulfillments.filter(accepted=True)
+              for fulfillment in fulfillments:


E111 indentation is not a multiple of four

stickler-ci · 2018-05-30T02:21:06Z

app/dashboard/models.py

+              fulfillments = bounty.fulfillments.filter(accepted=True)
+              for fulfillment in fulfillments:
+                if isinstance(fulfillment, BountyFulfillment):
+                  funded_bounty_fulfillments.append(fulfillment) 


E111 indentation is not a multiple of four
W291 trailing whitespace

stickler-ci · 2018-05-30T02:21:08Z

app/marketing/mails.py

+            print("-----" * 100)
+            print(html)
+            print("-----" * 100)
+            from_email = settings.PERSONAL_CONTACT_EMAIL


F841 local variable 'from_email' is assigned to but never used

stickler-ci · 2018-05-30T02:21:08Z

app/marketing/mails.py

+            print("-----" * 100)
+            from_email = settings.PERSONAL_CONTACT_EMAIL
+
+            #if not should_suppress_notification_email(to_email, 'roundup'):


E265 block comment should start with '# '

stickler-ci · 2018-05-30T02:21:08Z

app/marketing/mails.py

+            from_email = settings.PERSONAL_CONTACT_EMAIL
+
+            #if not should_suppress_notification_email(to_email, 'roundup'):
+                #send_mail(from_email, to_email, subject, text, html, from_name="Kevin Owocki (Gitcoin.co)")


E116 unexpected indentation (comment)
E265 block comment should start with '# '

stickler-ci · 2018-05-30T02:21:09Z

app/marketing/management/commands/send_quarterly_stats.py

+
+        platform_wide_stats = get_platform_wide_stats()
+
+<<<<<<< HEAD


E305 expected 2 blank lines after class or function definition, found 1
E999 SyntaxError: invalid syntax
E227 missing whitespace around bitwise or shift operator
E225 missing whitespace around operator

stickler-ci · 2018-05-30T02:21:09Z

app/marketing/management/commands/send_quarterly_stats.py

+        platform_wide_stats = get_platform_wide_stats()
+
+<<<<<<< HEAD
+        for counter, to_email in enumerate(email_list):


E113 unexpected indentation

stickler-ci · 2018-05-30T02:22:04Z

app/marketing/mails.py

@@ -323,6 +323,33 @@ def new_match(to_emails, bounty, github_username):
        translation.activate(cur_language)


+def quarterly_stats(to_emails=None):
+    if not platform_wide_stats:


F821 undefined name 'platform_wide_stats'

stickler-ci · 2018-05-30T02:22:04Z

app/marketing/mails.py

+            print("-----" * 100)
+            print(html)
+            print("-----" * 100)
+            from_email = settings.PERSONAL_CONTACT_EMAIL


F841 local variable 'from_email' is assigned to but never used

stickler-ci · 2018-05-30T02:22:05Z

app/marketing/management/commands/send_quarterly_stats.py

+    along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+'''
+import logging


F401 'logging' imported but unused

stickler-ci · 2018-06-01T02:47:50Z

app/dashboard/models.py

+            'user_total_earned_eth': Total earnings of user in ETH.
+            'user_total_earned_usd': Total earnings of user in USD.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-01T02:47:50Z

app/dashboard/models.py

+            'user_active_in_last_quarter': bool, if the user was active in last quarter
+            'user_no_of_languages': No of languages user used while working on bounties.
+            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

stickler-ci · 2018-06-01T02:47:50Z

app/dashboard/models.py

+                user_funded_bounty_developers.append('@' + fulfillment.fulfiller_github_username.lstrip('@'))
+            user_funded_bounty_developers = [*{*user_funded_bounty_developers}]
+            if funded_fulfillments_with_hours_counted:
+                avg_hourly_rate_per_funded_bounty = float(total_funded_hourly_rate) / float(funded_fulfillments_with_hours_counted)


E501 line too long (131 > 120 characters)

oogetyboogety · 2018-06-01T04:16:38Z

thelostone-mc · 2018-06-02T18:00:38Z

Ah looks like something went wrong during your rebase. This PR has commits pushed in by other folks ! Could you cherry-pick them and ensure the PR just has your commits ? ^_^

stickler-ci · 2018-06-02T23:19:35Z

app/dashboard/models.py

+        user_active_in_last_quarter = False
+        user_fulfilled_bounties = False
+        user_funded_bounties = False
+        last_quarter = datetime.now() - timedelta(days=90)


F821 undefined name 'timedelta'

oogetyboogety · 2018-06-02T23:20:50Z

@thelostone-mc Done! Thanks, please provide any further instructions I can complete to get this merged!

Thanks again

stickler-ci · 2018-06-03T01:47:24Z

app/dashboard/models.py

@@ -1194,6 +1194,187 @@ def get_quarterly_stats(self):
            'user_languages': user_languages
        }

+    @property


C901 'Profile.get_quarterly_stats' is too complex (17)
F811 redefinition of unused 'get_quarterly_stats' from line 1122

codecov · 2018-06-03T01:50:32Z

Codecov Report

Merging #1311 into master will decrease coverage by 1.19%.
The diff coverage is 29.68%.

@@            Coverage Diff            @@
##           master    #1311     +/-   ##
=========================================
- Coverage   30.63%   29.44%   -1.2%     
=========================================
  Files         124      132      +8     
  Lines        8783     9847   +1064     
  Branches     1141     1284    +143     
=========================================
+ Hits         2691     2899    +208     
- Misses       5984     6842    +858     
+ Partials      108      106      -2

Impacted Files	Coverage Δ
app/retail/emails.py	`20.79% <100%> (-2.12%)`	⬇️
app/dashboard/models.py	`52.51% <28.57%> (-3.07%)`	⬇️
app/gas/utils.py	`70.96% <0%> (-13.65%)`	⬇️
app/retail/utils.py	`15.19% <0%> (-6.55%)`	⬇️
app/app/urls.py	`89.74% <0%> (-4.55%)`	⬇️
app/app/sitemaps.py	`68.62% <0%> (-3.88%)`	⬇️
app/dashboard/signals.py	`54.54% <0%> (-3.79%)`	⬇️
app/dashboard/router.py	`31.42% <0%> (-3.31%)`	⬇️
app/app/settings.py	`80.43% <0%> (-2.72%)`	⬇️
... and 35 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c23fccc...083d5ce. Read the comment docs.

stickler-ci · 2018-06-03T02:02:16Z

app/dashboard/models.py

@@ -1127,22 +1090,97 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-03T02:02:16Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

There is no recommendation on https://www.python.org/dev/peps/pep-0257/, could you advise

stickler-ci · 2018-06-03T02:02:16Z

app/dashboard/models.py

@@ -1743,6 +1591,9 @@ def i18n_description(self):
    def i18n_link_copy(self):
        return _(self.link_copy)

+    def __str__(self):


F811 redefinition of unused 'str' from line 1545

…under stats and matched bounties for inactive users

… quarterly_status method dictionary, inactive user matched bounties from quarterly_status method dictionary

…s method

…form quarterly stats method

owocki · 2018-06-05T00:06:27Z

looking at this now

owocki · 2018-06-05T00:07:34Z

app/dashboard/models.py

        last_quarter = datetime.now() - timedelta(days=90)
        bounties = self.bounties.filter(modified_on__gte=last_quarter)
        fulfilled_bounties = [
            bounty for bounty in bounties if bounty.is_hunter(self.handle) and bounty.status == 'done'
        ]
        fulfilled_bounties_count = len(fulfilled_bounties)
+        funded_bounties = [
+            bounty for bounty in bounties if bounty.is_funder(self.handle)


is any of this data STUBBed out? are you unsure of any of the calcs you did? if so, mind throwing a few TODOs in there for us to work through?

Added TODOs

owocki · 2018-06-05T00:08:13Z

app/dashboard/models.py

+                relevant_bounties = Bounty.objects.none()
+                for keyword in user_coding_languages:
+                    relevant_bounties = relevant_bounties.union(potential_bounties.filter(
+                            network='rinkeby',


should not hardcode this network

owocki · 2018-06-05T00:12:04Z

over all looking good! heres my testing notes

double @ signs on this module http://bits.owocki.com/3Y0b100V2W28/Screen%20Shot%202018-06-04%20at%206.10.39%20PM.png
rounding http://bits.owocki.com/1o2b0D3H0J2d/Screen%20Shot%202018-06-04%20at%206.11.18%20PM.png

could you post screenshots from each of the personas pls? that'll make testing/QA easier for everyone else

cc @mbeacom @thelostone-mc in case anyone else wants to QA too

stickler-ci · 2018-06-05T22:52:46Z

app/dashboard/models.py

+                user_funded_bounty_developers.append(fulfillment.fulfiller_github_username.lstrip('@'))
+            user_funded_bounty_developers = [*{*user_funded_bounty_developers}]
+            if funded_fulfillments_with_hours_counted:
+                avg_hourly_rate_per_funded_bounty = float(total_funded_hourly_rate) / float(funded_fulfillments_with_hours_counted)


E501 line too long (131 > 120 characters)

stickler-ci · 2018-06-05T23:01:47Z

app/dashboard/models.py

+                for bounty in funded_bounties
+            ])
+            total_funded_hourly_rate = float(0)
+            #TODO: Explain to the user that we only counted Bounties that developers entered # of hours worked on


E265 block comment should start with '# '

Do you need us to finish the BE for these TODOs?

i just resolved this TODO

stickler-ci · 2018-06-05T23:01:48Z

app/dashboard/models.py

+                    hourly_rate_bounties_counted += 1
+            funded_bounty_fulfillments = []
+            for bounty in funded_bounties:
+                #TODO: Check that all bounty fulfillments have acceptance and that there are no bounties that have skipped the Acceptance in fulfillment step of workflow


E265 block comment should start with '# '
E501 line too long (169 > 120 characters)

stickler-ci · 2018-06-05T23:01:48Z

app/dashboard/models.py

+                avg_hourly_rate_per_funded_bounty = 0
+                avg_hours_per_funded_bounty = 0
+            funded_fulfilled_bounties = [
+                #TODO: Change this to check for terminal statuses instead?


E265 block comment should start with '# '

stickler-ci · 2018-06-05T23:01:48Z

app/dashboard/models.py

+                            current_bounty=True,
+                            metadata__icontains=keyword,
+                            idx_status__in=['open'],
+                            #TODO: Identify the random sampling method used, see if it sending mass emails becomes slow


E265 block comment should start with '# '

stickler-ci · 2018-06-05T23:01:48Z

app/dashboard/models.py

+                            metadata__icontains=keyword,
+                            idx_status__in=['open'],
+                            #TODO: Identify the random sampling method used, see if it sending mass emails becomes slow
+                            ).order_by('?') 


W291 trailing whitespace

oogetyboogety · 2018-06-05T23:16:55Z

@owocki Done, please review!

stickler-ci · 2018-06-13T23:15:33Z

app/dashboard/models.py

@@ -1127,22 +1127,97 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-13T23:15:33Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

stickler-ci · 2018-06-13T23:15:33Z

app/dashboard/models.py

+                user_funded_bounty_developers.append(fulfillment.fulfiller_github_username.lstrip('@'))
+            user_funded_bounty_developers = [*{*user_funded_bounty_developers}]
+            if funded_fulfillments_with_hours_counted:
+                avg_hourly_rate_per_funded_bounty = float(total_funded_hourly_rate) / float(funded_fulfillments_with_hours_counted)


E501 line too long (131 > 120 characters)

stickler-ci · 2018-06-13T23:15:33Z

app/dashboard/models.py

+                            current_bounty=True,
+                            metadata__icontains=keyword,
+                            idx_status__in=['open'],
+                            ).order_by('?') 


W291 trailing whitespace

owocki

just tested this and pushed up a few commits. this has my 👍 after you resolve @mbeacom 's concerns @oogetyboogety !

stickler-ci · 2018-06-13T23:18:50Z

app/dashboard/models.py

@@ -1127,22 +1127,97 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-13T23:18:50Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

stickler-ci · 2018-06-13T23:18:50Z

app/dashboard/models.py

+                user_funded_bounty_developers.append(fulfillment.fulfiller_github_username.lstrip('@'))
+            user_funded_bounty_developers = [*{*user_funded_bounty_developers}]
+            if funded_fulfillments_with_hours_counted:
+                avg_hourly_rate_per_funded_bounty = float(total_funded_hourly_rate) / float(funded_fulfillments_with_hours_counted)


E501 line too long (131 > 120 characters)

…-email

stickler-ci · 2018-06-19T01:40:24Z

app/dashboard/models.py

@@ -1127,22 +1127,95 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-19T01:40:25Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

stickler-ci · 2018-06-19T01:40:25Z

app/dashboard/models.py

+                avg_hourly_rate_per_funded_bounty = \
+                    float(total_funded_hourly_rate) / float(funded_fulfillments_with_hours_counted)
+                avg_hours_per_funded_bounty = \
+                        float(total_funded_hours) / float(funded_fulfillments_with_hours_counted)


E127 continuation line over-indented for visual indent

stickler-ci · 2018-06-20T04:38:57Z

app/dashboard/models.py

@@ -1127,22 +1127,95 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-06-20T04:38:57Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

oogetyboogety · 2018-06-20T04:45:55Z

@owocki @mbeacom Fixed most of the review comments, can you look over once more

mbeacom · 2018-06-25T14:48:17Z

@oogetyboogety Would you mind posting screenshots or gifs for these changes?

owocki · 2018-07-03T20:33:27Z

@oogetyboogety we are keen to move fowrward on this since its now Q3! let us know what you think of @mbeacom 's question above

oogetyboogety · 2018-07-04T15:35:22Z

@owocki I'll get those wrapped up and uploaded today!

…of passing the whole array for total_funded_usd

stickler-ci · 2018-07-05T04:40:46Z

app/dashboard/models.py

@@ -1127,22 +1127,95 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-07-05T04:40:46Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

oogetyboogety · 2018-07-05T04:41:51Z

stickler-ci · 2018-07-05T04:46:53Z

app/dashboard/models.py

@@ -1127,22 +1127,95 @@ def get_quarterly_stats(self):
            dict : containing the following information
            'user_total_earned_eth': Total earnings of user in ETH.
            'user_total_earned_usd': Total earnings of user in USD.
-            'user_fulfilled_bounties_count': Total bounties fulfilled by user.
+            'user_total_funded_usd': Total value of bounties funded by the user on bounties in done status in USD
+            'user_total_funded_hours': Total hours input by the developers on the fulfillment of bounties created by the user in USD


E501 line too long (132 > 120 characters)

stickler-ci · 2018-07-05T04:46:53Z

app/dashboard/models.py

            'user_active_in_last_quarter': bool, if the user was active in last quarter
            'user_no_of_languages': No of languages user used while working on bounties.
            'user_languages': Languages that were used in bounties that were worked on.
+            'relevant_bounties': a list of Bounty(s) that would match the skillset input by the user into the Match tab of their settings


E501 line too long (137 > 120 characters)

owocki · 2018-07-24T00:13:05Z

these look really good!

owocki · 2018-07-24T00:16:24Z

@oogetyboogety can u pls request payout via the gitcoin bounty?

stickler-ci reviewed May 30, 2018

View reviewed changes

oogetyboogety mentioned this pull request May 30, 2018

quarterly stats v2 -- funder and "no action" views #1036

Closed

stickler-ci reviewed Jun 1, 2018

View reviewed changes

oogetyboogety changed the title ~~WIP Stats email v2~~ Stats email v2 Jun 1, 2018

stickler-ci reviewed Jun 2, 2018

View reviewed changes

stickler-ci reviewed Jun 3, 2018

View reviewed changes

oogetyboogety added 7 commits June 2, 2018 22:12

Update get_quarterly_stats method on Django Profile model to return f…

8fb9fce

…under stats and matched bounties for inactive users

Updated quarterly stats email template with funder view of stats from…

7054f4e

… quarterly_status method dictionary, inactive user matched bounties from quarterly_status method dictionary

Cleanup of template to match names in dictionary from quarterly state…

5c28eb7

…s method

Cleanup of template to match hourly rate variable name in dictioanry …

18cbe4d

…form quarterly stats method

Add user languages back in after it was removed for some reason

fcf06c2

Cleanup of completion percent

7a59d39

Cleanup for user_langages that were added back in

55c00e2

owocki reviewed Jun 5, 2018

View reviewed changes

oogetyboogety added 2 commits June 5, 2018 18:36

Changed from hardcoded rinkeby test network to Profile.get_network()

0ae0190

Added rounding on average usd earned per bounty back

9a15407

stickler-ci reviewed Jun 5, 2018

View reviewed changes

Added TODOs for further questions as requested by @owocki

8e281a0

stickler-ci reviewed Jun 5, 2018

View reviewed changes

mbeacom changed the title ~~Stats email v2~~ WIP: Stats email v2 Jun 7, 2018

mbeacom added in progress labels Jun 7, 2018

resolving todos

509f82f

ghost assigned owocki Jun 13, 2018

stickler-ci reviewed Jun 13, 2018

View reviewed changes

smal lthing

42ddb96

owocki previously approved these changes Jun 13, 2018

View reviewed changes

stickler-ci reviewed Jun 13, 2018

View reviewed changes

oogetyboogety added 2 commits June 18, 2018 21:29

Added formatting changes, cleanup

e60686b

Merge branch 'stats-email' of github.com:oogetyboogety/web into stats…

491d09d

…-email

oogetyboogety dismissed owocki’s stale review via 491d09d June 19, 2018 01:39

stickler-ci reviewed Jun 19, 2018

View reviewed changes

Internationalization of email

bff6864

stickler-ci reviewed Jun 20, 2018

View reviewed changes

Fixed aggregation's output to the first element of the array instead …

5e8c0cc

…of passing the whole array for total_funded_usd

stickler-ci reviewed Jul 5, 2018

View reviewed changes

Cleanup

083d5ce

stickler-ci reviewed Jul 5, 2018

View reviewed changes

owocki merged commit 19b3693 into gitcoinco:master Jul 24, 2018

ghost removed the in progress label Jul 24, 2018

WIP: Stats email v2 #1311

WIP: Stats email v2 #1311

Conversation

oogetyboogety commented May 30, 2018 • edited Loading

Description

Checklist

Affected core subsystem(s)

Testing

Refers/Fixes

oogetyboogety commented May 30, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oogetyboogety commented Jun 1, 2018

thelostone-mc commented Jun 2, 2018

Choose a reason for hiding this comment

oogetyboogety commented Jun 2, 2018

Choose a reason for hiding this comment

codecov bot commented Jun 3, 2018 • edited Loading

Codecov Report

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

owocki commented Jun 5, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

owocki commented Jun 5, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oogetyboogety commented Jun 5, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

owocki left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oogetyboogety commented Jun 20, 2018

mbeacom commented Jun 25, 2018

owocki commented Jul 3, 2018 • edited Loading

oogetyboogety commented Jul 4, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oogetyboogety commented Jul 5, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

owocki commented Jul 24, 2018

owocki commented Jul 24, 2018

oogetyboogety commented May 30, 2018 •

edited

Loading

codecov bot commented Jun 3, 2018 •

edited

Loading

owocki commented Jul 3, 2018 •

edited

Loading