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

Remove Bity Rates from Local Storage #435

Merged
merged 3 commits into from
Nov 21, 2017
Merged

Remove Bity Rates from Local Storage #435

merged 3 commits into from
Nov 21, 2017

Conversation

james-prado
Copy link
Contributor

@james-prado james-prado commented Nov 21, 2017

Closes MyEtherWallet/MyEtherWallet#392

  • Replace swap.bityRates with initial value when storing in local storage
  • Change initial value of state.bityRates from {} to null

bityRates no longer persist in localstorage and are updated on each page load.

common/store.ts Outdated
@@ -111,12 +111,13 @@ const configureStore = () => {
languageSelection: state.config.languageSelection,
customNodes: state.config.customNodes
},
swap: state.swap,
swap: { ...state.swap, bityRates: swapInitialState.bityRates },
Copy link
Contributor

Choose a reason for hiding this comment

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

This still persists state.

bityRates: null is what you're looking for.

@dternyak dternyak changed the title Remove bityrates from Local Storage Remove Bity Rates from Local Storage Nov 21, 2017
@james-prado james-prado added the status: ready for review Open PR's that are ready for technical review. Replaced with "changes needed" or "ready to merge". label Nov 21, 2017
@dternyak dternyak merged commit d28f2ce into MyCryptoHQ:develop Nov 21, 2017
@james-prado james-prado deleted the remove-bityrates-from-ls branch January 15, 2018 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready for review Open PR's that are ready for technical review. Replaced with "changes needed" or "ready to merge".
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants