Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Retool the ledger updates into the 0.13.1-branch #6706

Merged
merged 3 commits into from
Jan 20, 2017

Conversation

mrose17
Copy link
Member

@mrose17 mrose17 commented Jan 17, 2017

Auditor @bsclifton

Fixes #6551
Fixes #6592

Test Plan

  1. go to about:preferences#payments
  2. set auto-suggest sites to off
  3. go to a site in another tab
  4. click on the BTC icon
  5. go back to the first tab and see if the site is present

@luixxiul
Copy link
Contributor

luixxiul commented Jan 18, 2017

Something seems to be wrong according to Travis: https://travis-ci.org/brave/browser-laptop/builds/192881925#L4354

@bsclifton
Copy link
Member

@luixxiul I believe the Chromedriver for latest version is not uploaded. We'll likely need new prebuilts

@bsclifton bsclifton force-pushed the 0.13.1-branch branch 2 times, most recently from d2028be to 9d038a4 Compare January 18, 2017 22:20
@mrose17
Copy link
Member Author

mrose17 commented Jan 19, 2017

@bsclifton - can we please get this into either master or 0.13.1 ? thanks!

@luixxiul
Copy link
Contributor

Rebase needed

Also fixes #6551 of the “sonorous seven”
To avoid currency exchange “jitter”.

Partially address #6707
@bsclifton bsclifton force-pushed the 0.13.1-ledger-updates branch from 7ee5856 to c5a1165 Compare January 19, 2017 22:48
@bsclifton bsclifton added this to the 0.13.1 milestone Jan 19, 2017
@bsclifton bsclifton self-requested a review January 20, 2017 00:00
Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

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

I re-ran tests which failed during CI, all except 1 passed. The one which kept failing appears to have been caused by an unrelated commit.

Ready for merge 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants