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

Grant fee rebase #4453

Merged
merged 14 commits into from
May 22, 2019
Merged

Grant fee rebase #4453

merged 14 commits into from
May 22, 2019

Conversation

danlipert
Copy link
Contributor

@danlipert danlipert commented May 22, 2019

Description

Rebased version of @thelostone-mc and I's grant fee implementation

Refers/Fixes

#4376

Testing

Tested manually - use grants/old to create a "v0" version of a grant to test how current grants with the v0 contract will work

@codecov
Copy link

codecov bot commented May 22, 2019

Codecov Report

Merging #4453 into master will decrease coverage by 0.33%.
The diff coverage is 14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4453      +/-   ##
==========================================
- Coverage   30.12%   29.79%   -0.34%     
==========================================
  Files         208      209       +1     
  Lines       16791    17266     +475     
  Branches     2255     2385     +130     
==========================================
+ Hits         5059     5144      +85     
- Misses      11537    11915     +378     
- Partials      195      207      +12
Impacted Files Coverage Δ
app/grants/admin.py 41.97% <ø> (ø) ⬆️
app/grants/urls.py 100% <ø> (ø) ⬆️
app/grants/abi.py 0% <0%> (ø) ⬆️
app/grants/models.py 50% <0%> (-0.72%) ⬇️
app/grants/convert_abi.py 0% <0%> (ø)
app/grants/views.py 13.92% <3.57%> (-0.79%) ⬇️
app/app/settings.py 78.65% <75%> (-0.12%) ⬇️
app/dashboard/views.py 15.3% <0%> (+1.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 86ae6ce...484c348. Read the comment docs.

@codecov
Copy link

codecov bot commented May 22, 2019

Codecov Report

Merging #4453 into master will increase coverage by 0.02%.
The diff coverage is 14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4453      +/-   ##
==========================================
+ Coverage   30.14%   30.16%   +0.02%     
==========================================
  Files         208      209       +1     
  Lines       16798    16846      +48     
  Branches     2257     2266       +9     
==========================================
+ Hits         5063     5081      +18     
- Misses      11538    11565      +27     
- Partials      197      200       +3
Impacted Files Coverage Δ
app/grants/admin.py 41.97% <ø> (ø) ⬆️
app/grants/urls.py 100% <ø> (ø) ⬆️
app/grants/abi.py 0% <0%> (ø) ⬆️
app/grants/models.py 50% <0%> (-0.72%) ⬇️
app/grants/convert_abi.py 0% <0%> (ø)
app/grants/views.py 13.92% <3.57%> (-0.79%) ⬇️
app/app/settings.py 78.65% <75%> (-0.12%) ⬇️
app/dashboard/utils.py 37.22% <0%> (+2.72%) ⬆️

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 10fd265...5b3d71a. Read the comment docs.

thelostone-mc and others added 4 commits May 22, 2019 19:03
- introduces section to let users choose % for gitcoin contribution
- summary reflects based on users selection
- code refactor
Copy link
Contributor

@SaptakS SaptakS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SaptakS SaptakS merged commit 9c6fdf6 into master May 22, 2019
@thelostone-mc thelostone-mc deleted the grant_fee_rebase branch July 4, 2019 14:53
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 this pull request may close these issues.

3 participants