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

grants/profile : add txn history template #2465

Merged
merged 6 commits into from
Oct 14, 2018

Conversation

thelostone-mc
Copy link
Member

Description

grants/profile: added txn history section …

  • UI template with be hardcoded
  • minimal refactoring
  • not mobile friendly (will fix that up in another PR )

x

Refers/Fixes

#2019

- set up route + view + empty template
- css clean up + indent fix
- renamed grants/grant_card -> grants/card
- split up card css to it's own file
- UI template with be hardcoded
- minimal refactoring
@thelostone-mc thelostone-mc added frontend This needs frontend expertise. grants labels Oct 14, 2018
@thelostone-mc thelostone-mc self-assigned this Oct 14, 2018
@thelostone-mc thelostone-mc requested a review from mbeacom October 14, 2018 19:26
history = [
{
'date': '16 Mar',
'value_true' : 1.0,

Choose a reason for hiding this comment

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

E203 whitespace before ':'

{
'date': '16 Mar',
'value_true' : 1.0,
'token_name' : 'ETH',

Choose a reason for hiding this comment

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

E203 whitespace before ':'

'date': '16 Mar',
'value_true' : 1.0,
'token_name' : 'ETH',
'frequency' : 'days',

Choose a reason for hiding this comment

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

E203 whitespace before ':'

'value_true' : 1.0,
'token_name' : 'ETH',
'frequency' : 'days',
'value_in_usdt_now' : 80,

Choose a reason for hiding this comment

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

E203 whitespace before ':'

},
{
'date': '24 April',
'value_true' : 90,

Choose a reason for hiding this comment

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

E203 whitespace before ':'

{
'date': '24 April',
'value_true' : 90,
'token_name' : 'DAI',

Choose a reason for hiding this comment

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

E203 whitespace before ':'

'date': '24 April',
'value_true' : 90,
'token_name' : 'DAI',
'frequency' : 'months',

Choose a reason for hiding this comment

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

E203 whitespace before ':'

'value_true' : 90,
'token_name' : 'DAI',
'frequency' : 'months',
'value_in_usdt_now' : 90,

Choose a reason for hiding this comment

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

E203 whitespace before ':'

@thelostone-mc thelostone-mc requested a review from SaptakS October 14, 2018 19:28
@codecov
Copy link

codecov bot commented Oct 14, 2018

Codecov Report

Merging #2465 into grants will increase coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           grants    #2465      +/-   ##
==========================================
+ Coverage   29.62%   29.65%   +0.03%     
==========================================
  Files         154      154              
  Lines       12115    12021      -94     
  Branches     1649     1607      -42     
==========================================
- Hits         3589     3565      -24     
+ Misses       8400     8338      -62     
+ Partials      126      118       -8
Impacted Files Coverage Δ
app/grants/views.py 30.43% <0%> (-0.45%) ⬇️
app/avatar/utils.py 18.28% <0%> (-3.21%) ⬇️
app/avatar/models.py 26.78% <0%> (+1.65%) ⬆️
app/bounty_requests/views.py 38.7% <0%> (+6.1%) ⬆️

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 0ee3802...c60ff60. Read the comment docs.

@mbeacom mbeacom merged commit d6c3020 into gitcoinco:grants Oct 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend This needs frontend expertise.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants