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/1464 li governance txs #1481

Merged
merged 10 commits into from
Nov 7, 2018
Merged

fedekunze/1464 li governance txs #1481

merged 10 commits into from
Nov 7, 2018

Conversation

fedekunze
Copy link
Contributor

Closes #1464

Description:

  • Adds governance txs to the tx history page

❤️ Thank you!


  • Added entries in CHANGELOG.md with issue # and GitHub username
  • Reviewed Files changed in the github PR explorer

@fedekunze fedekunze added the blocked ✋ issues blocked by other implementations/issues label Oct 23, 2018
@fedekunze
Copy link
Contributor Author

blocked by release of TmUI

@faboweb
Copy link
Collaborator

faboweb commented Oct 30, 2018

released 0.3.1

@faboweb faboweb removed the blocked ✋ issues blocked by other implementations/issues label Oct 30, 2018
@codecov
Copy link

codecov bot commented Nov 7, 2018

Codecov Report

Merging #1481 into develop will increase coverage by 0.07%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1481      +/-   ##
===========================================
+ Coverage    96.33%   96.41%   +0.07%     
===========================================
  Files           97       97              
  Lines         1800     1811      +11     
  Branches        86       86              
===========================================
+ Hits          1734     1746      +12     
+ Misses          56       55       -1     
  Partials        10       10
Impacted Files Coverage Δ
app/src/renderer/routes.js 100% <ø> (ø) ⬆️
...rer/components/governance/PageProposalsNewText.vue 20% <ø> (ø) ⬆️
...rc/renderer/components/wallet/PageTransactions.vue 97.14% <ø> (+2.54%) ⬆️
.../renderer/components/governance/PageGovernance.vue 100% <ø> (ø) ⬆️
app/src/renderer/connectors/lcdClient.js 97.56% <100%> (+0.12%) ⬆️
app/src/renderer/vuex/modules/transactions.js 100% <100%> (ø) ⬆️
app/src/renderer/connectors/lcdClientMock.js 99.39% <100%> (+0.02%) ⬆️

@fedekunze fedekunze changed the title WIP: Fedekunze/1464 li governance txs fedekunze/1464 li governance txs Nov 7, 2018
children: [
{
path: `/proposals`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah

const walletTxs = await dispatch(`getTx`, `wallet`)
commit(`setWalletTxs`, walletTxs)

const allTxs = stakingTxs.concat(walletTxs)
const allTxs = stakingTxs.concat(governanceTxs.concat(walletTxs))
Copy link
Collaborator

Choose a reason for hiding this comment

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

🐱 within a 🐱 😆

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jbibla jbibla merged commit f62a00d into develop Nov 7, 2018
@jbibla jbibla deleted the fedekunze/1464-li-gov-tx branch November 7, 2018 15:51
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.

Create LiTransaction for governance
3 participants