-
Notifications
You must be signed in to change notification settings - Fork 504
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
chore(redux): convert preferences to redux slice #6297
Conversation
@@ -7,23 +7,21 @@ import { selectors } from 'data' | |||
export const getData = createDeepEqualSelector( | |||
[ | |||
selectors.core.settings.getCurrency, | |||
selectors.preferences.getPriceChart, | |||
selectors.components.priceChart.getTime, |
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.
preferences.getPriceChart
was updated, via the preferences
reducer, with the same actions used in the priceChart
reducer - so in this case the data obtained was exactly the same
No New Or Fixed Issues Found |
@@ -78,7 +90,7 @@ class CryptoLinkHandlingContainer extends React.PureComponent { | |||
<a | |||
href='https://caniuse.com/?search=registerProtocolHandler' | |||
target='_blank' | |||
rel='noopener noreferrrer noreferrer' | |||
rel='noopener noreferrer' |
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.
nice 👍
82e26ff
to
4c8b569
Compare
I'm getting the slight idea that many of these are not used - as they clash with some things that are in blockchain-wallet-v4-frontend/src/data/modules/settings/reducers.js