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

Coloradocoin receive page (due Feb 1) #261

Closed
owocki opened this issue Jan 22, 2018 · 9 comments · Fixed by #331
Closed

Coloradocoin receive page (due Feb 1) #261

owocki opened this issue Jan 22, 2018 · 9 comments · Fixed by #331

Comments

@owocki
Copy link
Contributor

owocki commented Jan 22, 2018

What

We will be distributing 1200 of these

32136966-d2ec7722-bbd4-11e7-9d3b-028e589157d2

at ETHDenver.com next month.

On the back of them is a QR code which will lead to a link that'll allow them to redeem their physical coin for 1 COLO.. We are building the redemption page!

Functional spec

  1. User scans QR Code, which takes them to https://gitcoin.co/coin/redeem
    2a. They are greeted with a page that says "This coin has already been redeemed" -or-
    2b. They are greeted with a page that says "You've got 1 COLO! Enter a forwarding address". This page has a text input for the address and a submit button.
  2. Upon submit, a page in the backend receives the request, and uses web3py to transfer 1 COLO to the specified address.
  3. The frontend presents a confirmation page, which a link to the txid on etherscan

Backend specific considerations

  1. Please create a new table called CoinRedemption, which contains shortcodes which can be used to redeem COLO. Please also create a table called CoinRedemptionRequest which tracks the redemption requests.
  2. Please create a management command that can generate n CoinRedemptions. You can base it off of https://github.com/gitcoinco/web/blob/master/app/linkshortener/management/commands/create_shortlinks.py
  3. New url route: https://gitcoin.co/coin/redeem/(XXXXX) -- It is very important that this page is rate limited to prevent people from enumerating all keys.
  4. On the backend, you can assume you'll have a wallet that I've preloaded with 1200 COLO, which you can use to issue the ERC20 transfers upon form submission.

Design

Design MUST be mobile-first and mobile friendly. 99% of usage of this page will be from a mobile device.

Please base upon the design of https://gitcoin.co/tip/receive/

screencapture-gitcoin-co-tip-receive-1516635018941

@gitcoinbot
Copy link
Member

This issue now has a funding of 0.25 ETH (245.68 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
  • $14308.32 more Funded OSS Work Available at: https://gitcoin.co/explorer

@gitcoinbot
Copy link
Member

The funding of 0.25 ETH (245.68 USD) attached has been claimed by @eswarasai.

@eswarasai, 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.

@vs77bb
Copy link
Contributor

vs77bb commented Jan 23, 2018

Hi @eswarasai, mind providing an update here? Apologies if I'm out of the loop 🙂 cc @owocki

@eswarasai
Copy link
Contributor

@owocki @vs77bb -- I haven't started development on this yet. Still planning around the code changes to be done. I think Kevin had nicely mentioned all the Frontend & Backend tasks breakdown for this. So I'd say we already have a plan in place.

And also @thelostone-mc might work alongside with me to get this done at the earliest possible.
Aditya, please do add if I'm missing anything here or any other questions that you might have.

@vs77bb
Copy link
Contributor

vs77bb commented Jan 23, 2018

@eswarasai Sounds good! Thanks for the update 👍

@owocki
Copy link
Contributor Author

owocki commented Feb 1, 2018

@eswarasai how goes it

@eswarasai
Copy link
Contributor

@owocki -- I should be able to wrap things up within next 24 hours :)

@gitcoinco gitcoinco deleted a comment from gitcoinbot Feb 15, 2018
@owocki
Copy link
Contributor Author

owocki commented Feb 15, 2018

just paid this out.. thanks @eswarasai ! https://etherscan.io/tx/0x312017647c37242b6b66b143ab9de63da6bb23816086120aa216ddfe9cef8915

@thelostone-mc
Copy link
Member

@eswarasai treat \m/

thelostone-mc added a commit that referenced this issue May 24, 2021
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 a pull request may close this issue.

5 participants