Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Csv export management command to spit out activity report #157

Closed
owocki opened this issue Dec 21, 2017 · 27 comments · Fixed by #180
Closed

Csv export management command to spit out activity report #157

owocki opened this issue Dec 21, 2017 · 27 comments · Fixed by #180
Labels
easy-pickings This is an easy issue for the community.

Comments

@owocki
Copy link
Contributor

owocki commented Dec 21, 2017

Requirements

i would like to create a management command that emails a CSV file to settings. CONTACT_EMAIL ..

the email subject should be Gitcoin Activity report from YYYY/MM/DD to YYYY/MM/DD

This management command should take a start_date and a end_date in YYYY/MM/DD format.

CSV Spec

This CSV file should have the following columns

  • Type (tip or bounty)
  • Created Datetime
  • Last Activity Datetime
  • Amount (ex: 1.5)
  • Denomination (ex: ETH)
  • Amount_in_ETH
  • Amount_in_USD
  • From_ETH_Address
  • To_ETH_Address
  • Repo_Name (Truffle, Metamask, etc)
  • From_Github_Username
  • To_Github_Username
  • Status (Received, Open, Active, Fulfilled )
  • Comments (e.g. Tip to @owocki, Fulfilled Bounty to @owocki for [PR LINK])

Please only include

  • tips/bounties that are on the mainnet
  • bounties that are current_bounty=True
  • that are within the specified timerange.

To submit bounty

  1. Raise a PR
  2. Include an example CSV file.
@owocki owocki added the easy-pickings This is an easy issue for the community. label Dec 21, 2017
@owocki owocki changed the title Csv export management command to spit out funding report Csv export management command to spit out activity report Dec 21, 2017
@gitcoinbot
Copy link
Member

This issue now has a funding of 0.15 ETH (119.54 USD) attached to it.

  • If you would like to work on this issue you can claim it here.
  • If you've completed this issue and want to claim the bounty you can do so here
  • Questions? Get help on the Gitcoin Slack
  • $12546.5 more Funded OSS Work Available at: https://gitcoin.co/explorer

@sc0Vu
Copy link
Contributor

sc0Vu commented Dec 21, 2017

It's cool, and I got few questions about this:
Does this command run by bounty users or serve admin?
Should we need to use date time picker?

@gitcoinbot
Copy link
Member

The funding of 0.15 ETH (119.47 USD) attached has been claimed by @mbeacom.

@mbeacom, please leave a comment to let the funder (@owocki) and the other parties involved your implementation plan. If you don't leave a comment, the funder may expire your claim at their discretion.

@owocki
Copy link
Contributor Author

owocki commented Dec 21, 2017

its a management command, not a website.. its run by server admin

@owocki
Copy link
Contributor Author

owocki commented Dec 27, 2017

@mbeacom do you still want this one? no pressure eitehr way, but it might be worth releasing it back into the wild for someone else to tackle (cc @sc0Vu ) if its not on ur frontburner

@mbeacom
Copy link
Contributor

mbeacom commented Dec 27, 2017

@owocki I can release it back. I have a base that I started on for this in a stash that I can provide as well.

@owocki
Copy link
Contributor Author

owocki commented Dec 27, 2017

cool!

@choochootrain
Copy link
Contributor

@owocki i can take this on. @mbeacom could you share the stash you started? if not i'll start without it and PR what i have

@choochootrain
Copy link
Contributor

i went ahead and PR'd what i have 🤷‍♂️

  • some of the csv fields don't make sense or are ambiguous for tips - please advise on what you want here
  • i don't have any test data to play with so if you could get me a fixture or a small amount of sample data i can do some more extensive testing

@owocki
Copy link
Contributor Author

owocki commented Dec 28, 2017

some of the csv fields don't make sense or are ambiguous for tips - please advise on what you want here

@choochootrain -- just use your best judgement for now. ill evolve later if needed

i don't have any test data to play with so if you could get me a fixture or a small amount of sample data i can do some more extensive testing

ok, sure. does a postgres dump work? mind DMing me on slack and ill hook up an export of the bounties/tips tables (minus any confidential info) for you

@owocki
Copy link
Contributor Author

owocki commented Jan 2, 2018

@choochootrain should i be testing this now?

btw, mind claiming the issue at https://gitcoin.co/funding/details?url=https://github.com/gitcoinco/web/issues/157 so i can payout once merged?

@choochootrain
Copy link
Contributor

@owocki sorry, yeah its ready to test! uploads a link with a 30 day expiration to access the csv. i'll rebase against master as well.

@choochootrain
Copy link
Contributor

^ also attempting to claim that issue gives me a This issue is no longer active. Please leave a comment here if you need help. message. looks like the bounty still hasn't expired so not sure what the deal is. i can create a new issue for this if you want, since the linked issue (#169) seems unrelated.

@gitcoinbot
Copy link
Member

The funding of 0.15 ETH (130.8 USD) attached has been claimed by @Mitch_Kosowski.

@Mitch_Kosowski, please leave a comment to let the funder (@owocki) and the other parties involved your implementation plan. If you don't leave a comment, the funder may expire your claim at their discretion.

@owocki
Copy link
Contributor Author

owocki commented Jan 3, 2018

@Mitch_Kosowski im gonna reject your claim. nothing person... its just that @choochootrain has already done a ton of work here :)

@mkosowsk
Copy link

mkosowsk commented Jan 3, 2018

Whoa, that's really weird. I swear I didn't claim this issue! 😳😳😳

So no worries about rejecting my claim as I don't think I did and if I did I didn't mean to!

The only way you can claim an issue is via gitcoin.co or directly on GitHub, right? I'm really not sure how I inadvertently claimed this 🤔

@owocki
Copy link
Contributor Author

owocki commented Jan 3, 2018

hmm... weird.. the claim came from https://etherscan.io/address/0xdbcc1bfaacf822071b011c5e7d842617b23d27c1 -- is that you @mkosowsk

yes, you can only claim via gitcoin.co... although we are working on integrating a github bot

@mkosowsk
Copy link

mkosowsk commented Jan 4, 2018

Yep, that's me but I don't have a Sent in my MetaMask from 9 hours ago which I believe I would have if I sent the claim in?

screen shot 2018-01-03 at 9 21 06 pm

I think this may be a bug as I'm 99.99% sure I didn't send this claim in (unless I entered a fugue state!). My last issue I claimed was premature so I wanted to only claim my next issue when I was ready for a PR, and this was sure the opposite lol 😂

@owocki
Copy link
Contributor Author

owocki commented Jan 4, 2018

@mkosowsk looks liek the claim was from 4 days ago.. https://etherscan.io/tx/0xe0d03a261a5851b3154399adeaf54c0a9f0b50fbc812559178122c3976b584a6

the gitcoin app just didnt process the tx until 9 (now 11) hours ago

@mkosowsk
Copy link

mkosowsk commented Jan 4, 2018

I see, thanks. That refreshes my memory, I did try to claim this a few days back but didn't see it go through... assumed I did something wrong when submitting the claim👍🏻

For my knowledge going forward, when a claim is submitted the Gitcoin app will process the tx generally pretty quickly or is there a bit of time built into this? Thanks!

@owocki
Copy link
Contributor Author

owocki commented Jan 4, 2018

it depends upon the gas price you set and also upon the network conditions at the time

here is an issue im coding up to make it clearer when the tx will confirm #205

@mkosowsk
Copy link

mkosowsk commented Jan 4, 2018

Ah that makes sense, think I set gas price too damn low so it probably took a while

Thanks for heads up on that ticket, looks like interesting stuff! Will walk through it to get some good understand when I get some free time 👍🏻👍🏻👍🏻

@owocki
Copy link
Contributor Author

owocki commented Jan 4, 2018

@choochootrain the rejection went through overnight. want to claim the issue now!?

@choochootrain
Copy link
Contributor

claiming with a low gas price, lets see if it goes through before i accidentally close the tab :)

@owocki
Copy link
Contributor Author

owocki commented Jan 5, 2018

even if u do close the tab, let me know and i can still process the tx on the backend

@gitcoinbot
Copy link
Member

The funding of 0.15 ETH (149.85 USD) attached has been claimed by @choochootrain.

@choochootrain, please leave a comment to let the funder (@owocki) and the other parties involved your implementation plan. If you don't leave a comment, the funder may expire your claim at their discretion.

@gitcoinbot
Copy link
Member

The funding of 0.15 ETH (163.17 USD) attached to this issue has been approved & issued to @choochootrain.

ethikz pushed a commit to ethikz/web that referenced this issue Jan 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy-pickings This is an easy issue for the community.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants