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

Fedekunze/1010 lcd stake #1068

Merged
merged 32 commits into from
Aug 10, 2018
Merged

Fedekunze/1010 lcd stake #1068

merged 32 commits into from
Aug 10, 2018

Conversation

fedekunze
Copy link
Contributor

Description:
Updates LCD endpoints for staking.
DO NOT MERGE until #cosmos-sdk/1880 has been merged !

❤️ Thank you!

@fedekunze fedekunze added DO NOT MERGE blocked ✋ issues blocked by other implementations/issues labels Aug 2, 2018
@fedekunze fedekunze self-assigned this Aug 2, 2018
@faboweb
Copy link
Collaborator

faboweb commented Aug 2, 2018

We use the prefix "WIP" for our PRs that are not ready to merge. Changed it for you.

@faboweb faboweb changed the title Fedekunze/1010 lcd stake WIP: Fedekunze/1010 lcd stake Aug 2, 2018
// },

// Get a list containing all the validator candidates
getValidators: req("GET", "/stakes/validators/"),
Copy link
Collaborator

@faboweb faboweb Aug 2, 2018

Choose a reason for hiding this comment

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

typo "stakes" instead of "stake"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

for some reason my ESlint wasn't working. I had to delete it and reinstall

@fedekunze fedekunze removed the blocked ✋ issues blocked by other implementations/issues label Aug 8, 2018
@codecov
Copy link

codecov bot commented Aug 8, 2018

Codecov Report

Merging #1068 into develop will decrease coverage by 1.13%.
The diff coverage is 78.64%.

@@             Coverage Diff             @@
##           develop    #1068      +/-   ##
===========================================
- Coverage    97.18%   96.05%   -1.14%     
===========================================
  Files           79       81       +2     
  Lines         1525     1595      +70     
  Branches        77       79       +2     
===========================================
+ Hits          1482     1532      +50     
- Misses          36       56      +20     
  Partials         7        7
Impacted Files Coverage Δ
app/src/renderer/components/common/AppHeader.vue 93.75% <ø> (ø) ⬆️
app/src/renderer/vuex/getters.js 100% <ø> (ø) ⬆️
app/src/renderer/vuex/modules/wallet.js 95.31% <ø> (-0.08%) ⬇️
app/src/renderer/vuex/modules/delegates.js 100% <100%> (ø) ⬆️
app/src/renderer/components/staking/PageBond.vue 100% <100%> (ø) ⬆️
...rc/renderer/components/wallet/PageTransactions.vue 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/user.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/validators.js 100% <100%> (ø) ⬆️
app/src/renderer/connectors/lcdClientMock.js 97.43% <100%> (+2.24%) ⬆️
app/src/renderer/vuex/modules/delegation.js 100% <100%> (ø) ⬆️
... and 7 more

@faboweb
Copy link
Collaborator

faboweb commented Aug 9, 2018

blocked on cosmos/cosmos-sdk#1949

@faboweb faboweb added the blocked ✋ issues blocked by other implementations/issues label Aug 9, 2018
@faboweb faboweb changed the title WIP: Fedekunze/1010 lcd stake Fedekunze/1010 lcd stake Aug 9, 2018
@faboweb faboweb removed the blocked ✋ issues blocked by other implementations/issues label Aug 10, 2018
@faboweb
Copy link
Collaborator

faboweb commented Aug 10, 2018

Reviewed and approved by @fedekunze

@faboweb faboweb merged commit b012577 into develop Aug 10, 2018
@faboweb faboweb deleted the fedekunze/1010-lcd-stake branch August 10, 2018 15:37
@fedekunze fedekunze restored the fedekunze/1010-lcd-stake branch August 10, 2018 17:19
@fedekunze fedekunze deleted the fedekunze/1010-lcd-stake branch August 29, 2018 16:34
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.

2 participants