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

Payout crowdfund #4227

Merged
merged 5 commits into from
Apr 23, 2019
Merged

Payout crowdfund #4227

merged 5 commits into from
Apr 23, 2019

Conversation

octavioamu
Copy link
Contributor

@octavioamu octavioamu commented Apr 18, 2019

Description
  • Show each crowdfund value / token
  • Show list of users that have submitted work
  • Click on usernames to quick add it into the select without searching / create more rows if anyone empty (to avoid overwrite if the user already filled some)

https://share.vidyard.com/watch/RpmNXDFZf57z559Li8XgzH

Checklist
Affected core subsystem(s)
Refers/Fixes

Fixes: #4165

Testing and Sign-off
Contributor
  • Read and followed the Contributor Guidelines
  • Tested all changes locally
  • Verified existing functionality
  • Ran make test and everything passed!
Reviewer
  • Affirm contributor guidelines have been followed and requested changes made
  • CI tests and linting pass
  • No conflicts (migrations, files, etc)
  • Regression tested against staging or local deployment
Funder
  • Validated requested changes were made to specification
  • Bounty payout released to the contributor

@codecov
Copy link

codecov bot commented Apr 18, 2019

Codecov Report

Merging #4227 into master will increase coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4227      +/-   ##
==========================================
+ Coverage   30.28%   30.32%   +0.03%     
==========================================
  Files         209      209              
  Lines       16604    16601       -3     
  Branches     2222     2222              
==========================================
+ Hits         5029     5034       +5     
+ Misses      11391    11385       -6     
+ Partials      184      182       -2
Impacted Files Coverage Δ
app/dashboard/views.py 13.79% <0%> (-0.02%) ⬇️
app/app/urls.py 90% <0%> (ø) ⬆️
app/retail/views.py 29.66% <0%> (+0.06%) ⬆️
app/dashboard/models.py 55.99% <0%> (+0.25%) ⬆️
app/dashboard/utils.py 36.72% <0%> (+0.49%) ⬆️

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 df9bc8a...edcb188. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 18, 2019

Codecov Report

Merging #4227 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4227      +/-   ##
==========================================
- Coverage   30.24%   30.24%   -0.01%     
==========================================
  Files         209      209              
  Lines       16606    16607       +1     
  Branches     2223     2223              
==========================================
  Hits         5022     5022              
- Misses      11403    11404       +1     
  Partials      181      181
Impacted Files Coverage Δ
app/dashboard/views.py 13.77% <0%> (-0.02%) ⬇️

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 f87d85e...beb32ab. Read the comment docs.

@thelostone-mc thelostone-mc merged commit 75017c5 into master Apr 23, 2019
@thelostone-mc thelostone-mc deleted the payout-crowdfund branch April 23, 2019 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants