-
Notifications
You must be signed in to change notification settings - Fork 100
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
Fabo/0.33.0 #2225
Fabo/0.33.0 #2225
Conversation
Co-Authored-By: faboweb <frznhope@gmail.com>
…tion-format' into fabo/0.33.0
Codecov Report
@@ Coverage Diff @@
## develop #2225 +/- ##
===========================================
- Coverage 95.39% 94.66% -0.73%
===========================================
Files 109 109
Lines 2279 2288 +9
Branches 114 116 +2
===========================================
- Hits 2174 2166 -8
- Misses 92 109 +17
Partials 13 13
|
Codecov Report
@@ Coverage Diff @@
## develop #2225 +/- ##
===========================================
+ Coverage 95.4% 95.42% +0.01%
===========================================
Files 109 109
Lines 2287 2295 +8
Branches 114 116 +2
===========================================
+ Hits 2182 2190 +8
Misses 92 92
Partials 13 13
|
It's not. Merge conflict. I removed the file again.
Federico Kunze <notifications@github.com> schrieb am Sa., 9. März 2019,
19:31:
… ***@***.**** commented on this pull request.
------------------------------
In app/src/renderer/components/transactions/TmLiAnyTransaction.vue
<#2225 (comment)>:
> @@ -0,0 +1,100 @@
+<template>
We renamed this file to LiAnyTransaction. Why is this being changed ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2225 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFmO2U3DIPobRYRtOKy4c9NaAhMl1sr4ks5vU_4AgaJpZM4bmvpg>
.
|
@@ -159,11 +156,11 @@ export default ({ node }) => { | |||
|
|||
await dispatch(`sendTx`, { | |||
type: `postDelegation`, | |||
to: wallet.address, // TODO strange syntax | |||
to: session.address, // TODO strange syntax |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still strange syntax?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, just changed the source of the address
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the strange syntax? can we create an issue ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is already an issue around refactoring the send module #1828
@@ -36,7 +36,7 @@ | |||
"prepush": "bash ./tasks/changelog-changed-check.sh && yarn lint", | |||
"postcheckout": "yarn", | |||
"watch": "tasks/watch.sh", | |||
"nodes": "node ./tasks/testnet.js", | |||
"nodes": "node ./tasks/local-testnet/start.js", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
hmm I still get the error from undelegations: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Governance penalty parameter was removed and none of the actions work.
Also the distribution tx action from
req(`GET`, `/txs?action=withdraw_delegation_reward&delegator=${address}`)()
changed from withdraw_delegation_reward
to withdraw_delegator_reward
@fedekunze could you re-eval please? |
This is still failing: (re)delegation, unbonding, proposals, send, etc |
will fix this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's merge I update the nodes now
Need to check version again. The commit stated 0.32.0. Maybe need to upgrade again. |
Karoly was this manually tested ? |
Closes #2103
Updates to SDK 0.33.0
Improves on local testnet generation and usage. Always sends tokens to a fixed address: "cosmos1ek9cd8ewgxg9w5xllq9um0uf4aaxaruvcw4v9e"
Needs the backend to be updated
Description:
Thank you! 🚀
For contributor:
PENDING.md
with issue # and GitHub usernameFiles changed
in the github PR explorerFor reviewer: