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

bump version #812

Merged
merged 1 commit into from
Oct 25, 2018
Merged

bump version #812

merged 1 commit into from
Oct 25, 2018

Conversation

BelfordZ
Copy link
Contributor

@BelfordZ BelfordZ commented Oct 18, 2018

Fixes #798 #785 #784 #782 #781 #780 #779 #777 #775 #774 #770 #767 #766 #760 #738 #737 #736 #734 #733 #725 #719 #704 #692 #676 #670

Addresses parts of #688 - allows vault to be running already when the wallet starts

Probably missed some other issues that it fixed, Ill edit this later if that is the case.

This release addresses

  • performance issues when transaction history grows large.
  • memory leaks and growth in number of timers
  • Many bugs relating to switching networks (testnet to mainnet, local to remote, ...)
  • adds support for ethereum transaction url handling. Wallet will now respond to the ethereum:// protocol
  • Upgrades electron to latest 3.x.x - Many performance fixes at the lower level. Many security fixes as well.
  • Fixed all dependencies with known security issues
  • overhaul transaction sending to display better information about the data component.
  • Upgrade ledger multiple versions - pretty much overhauled all of the ledger handling code. Smoothed out many of the issues for ledger users in doing so.
  • No longer crashes if vault is already running
  • Send max works more reliably now -- though is still not perfect. The trouble is that we want to send the TX and get 0 change. Gasprice changes, and so its really up to how fast you can check the gasPrice before sending the tx. We will have a better solution for this in a later release.

@ghost ghost assigned BelfordZ Oct 18, 2018
@ghost ghost added the review label Oct 18, 2018
@codecov
Copy link

codecov bot commented Oct 18, 2018

Codecov Report

Merging #812 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #812   +/-   ##
=======================================
  Coverage   26.15%   26.15%           
=======================================
  Files         158      158           
  Lines        3028     3028           
  Branches      406      406           
=======================================
  Hits          792      792           
  Misses       2004     2004           
  Partials      232      232

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 a757d01...b30ebf8. Read the comment docs.

Copy link
Contributor

@whilei whilei left a comment

Choose a reason for hiding this comment

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

💚

@BelfordZ BelfordZ merged commit cfb48df into master Oct 25, 2018
@ghost ghost removed the review label Oct 25, 2018
@splix splix deleted the bump branch April 8, 2019 02:06
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.

Failed to fetch
3 participants